Skip to content
Snippets Groups Projects
Unverified Commit 02ed7de8 authored by emailweixu's avatar emailweixu Committed by GitHub
Browse files

Update contributing.rst about git pull (#1429)

* Update contributing.rst about git pull

For new version of git,  the default `git pull` will rebase, which is what we want to avoid. So we need to update the instruction.

* Update contributing.rst
parent 77cdad8b
No related branches found
No related tags found
No related merge requests found
......@@ -64,7 +64,11 @@ as NEW commit to your previous commits. Do not use :code:`--amend` option for th
commit, because then you will have to use :code:`-f` option to push your change to
github and review will be more difficult because the new change cannot
be separated from previous change. For the same reason, if you need to incorporate
the latest code from master, please use :code:`git pull` instead of :code:`git pull --rebase`.
the latest code from master, please avoid rebase. If your code is rebased, you won't
be able to push your PR without using `-f` option. So you should use :code:`git pull --rebase=false`
instead of :code:`git pull`. You can globally change the default behavior of :code:`git pull`
to not rebase by setting :code:`git config --global pull.rebase false` so that you can simply
use :code:`git pull`.
Coding standard
---------------
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment