[PATCH] D80088: [llvm][docs] Add step by step git to GettingStarted

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 18 13:33:54 PDT 2020


spatel added inline comments.


================
Comment at: llvm/docs/GettingStarted.rst:511
+  # Check that the list of commits about to be pushed is correct.
+  % git log origin/master...HEAD --oneline
+  # Push to Github.
----------------
Thanks - that's basically the feature of 'git llvm push' that was most useful to me.
It did some variation of this:
```
$ git llvm push
`git fetch https://github.com/llvm/llvm-project.git master` printed to stderr:
>From https://github.com/llvm/llvm-project
 * branch                    master     -> FETCH_HEAD
Pushing 2 commits:
  f29a9387ace [x86] add tests for disguised horizontal ops; NFC
  f40b5065511 [x86] add tests for heroic horizontal ops; NFC
Are you sure you want to create 2 commits? (y/N): n
```

That 'Are you sure...' pause is what kept me out of trouble, so I'd probably just end up creating a weak version of the old script again.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80088/new/

https://reviews.llvm.org/D80088





More information about the llvm-commits mailing list