[PATCH] D147284: docs: Document procedure for updating pull requests

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 30 23:58:47 PDT 2023


jhenderson added inline comments.


================
Comment at: llvm/docs/GitHub.rst:28
+to pull requests in the future.  This section documents the pull request
+policies will be adopting once the project starts using them.
+
----------------



================
Comment at: llvm/docs/GitHub.rst:37-40
+If you choose to do this, you must squash and merge before committing and
+you must use the pull request title and description as the commit message.
+This will allow reviewers to review the commit message before approving the
+commit.
----------------
I agree with the idea of being able to push "fix up" commits to the PR and using Squash & Merge. It might be worth emphasising that the default final commit message will match exactly the PR title and body. I've been known to include additional comments in my initial PR description, to help reviewers, that don't necessarily want to be part of the final commit message, and I'm sure others have too.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147284



More information about the llvm-commits mailing list