[PATCH] D71640: [docs] Remove `git llvm push` and `git llvm revert` from GettingStarted
Mehdi AMINI via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 17 18:45:54 PST 2019
mehdi_amini added inline comments.
================
Comment at: llvm/docs/GettingStarted.rst:494
+Once a patch is reviewed, you should rebase it, re-test locally, and commit the
+changes to LLVM's master branch. This is done using `git push`.
----------------
You're using "commit" in a fairly liberal way. Since we're native Git now, can we adjust the vocabulary?
```
Once a patch is reviewed, you can commit it in your local clone of the repository in the master branch, and use `git push` to push it to the upstream repository on GitHub. You may need to run `git pull --rebase` in case other changes have been pushed in the meantime.
Note that LLVM currently has a linear-history policy, which means that merge commits are
not allowed. The `llvm-project` repository on GitHub is configured to reject pushes
that include merges, so make sure to use `--rebase` when you pull. Alternatively you can
do once `git config --global pull.rebase true` to make it the default on your system.
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71640/new/
https://reviews.llvm.org/D71640
More information about the llvm-commits
mailing list