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

[Tom Stellard via Phabricator via llvm-commits]

tstellar created this revision.
Herald added a project: All.
tstellar requested review of this revision.
Herald added a project: LLVM.

Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D147284

Files:
  llvm/docs/GitHub.rst


Index: llvm/docs/GitHub.rst
===================================================================
--- llvm/docs/GitHub.rst
+++ llvm/docs/GitHub.rst
@@ -19,6 +19,26 @@
 the future if needed to support "stacked" pull request, but in that case only
 branches being used for "stacked" pull requests will be allowed.
 
+Pull Requests
+=============
+The LLVM Project does not currently accept pull requests for the llvm/llvm-project
+repository.  However, there is a
+`plan <https://discourse.llvm.org/t/code-review-process-update/63964>`_ to move
+to pull requests in the future.  This section documents the pull request
+policies will be adopting once the project starts using them.
+
+
+Updating Pull Requests
+^^^^^^^^^^^^^^^^^^^^^^
+When updating a pull request, you may push additional "fix up" commits to
+your branch instead of force pushing.  This makes it easier for github to
+track the context of previous review comments.
+
+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.
+
 
 Releases
 ========


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D147284.509845.patch
Type: text/x-patch
Size: 1191 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230330/c5fb95d0/attachment.bin>