[llvm] [Docs] Update documentation for the new GitHub workflow (PR #65162)

Nicolai Hähnle via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 1 11:57:49 PDT 2023


================
@@ -46,11 +73,106 @@ The default commit message for a squashed pull request is the pull request
 description, so this will allow reviewers to review the commit message before
 approving the commit.
 
+When pushing to your branch, make sure you push to the correct fork. Check your
+remotes with:
+
+::
+
+  git remote -v
+
+And make sure you push to the remote that's pointing to your fork.
+
+Rebasing Pull Requests and Force Pushes
+---------------------------------------
+In general, you should avoid rebasing a Pull Request and force pushing to the
+branch that's the root of the Pull Request during the review. This action will
+make the context of the old changes and comments harder to find and read.
----------------
nhaehnle wrote:

I think *avoiding* rebases is a good policy (though I'd encourage folks to try my [diff-modulo-base](https://git.sr.ht/~nhaehnle/diff-modulo-base) tool to help in reviews), but it should be *allowed*.

That most closely reflects how things work with Phabricator.

https://github.com/llvm/llvm-project/pull/65162


More information about the llvm-commits mailing list