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

[Tom Stellard via Phabricator via llvm-commits]

tstellar updated this revision to Diff 516609.
tstellar added a comment.

Add documentation about creating pull requests from a personal fork.


Repository:
  rG LLVM Github Monorepo

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

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,31 @@
 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.
+
+Creating Pull Requests
+^^^^^^^^^^^^^^^^^^^^^^
+For pull requests, please push a branch to your fork of the llvm-project and
+`create a pull request from the fork <https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork>`_.
+
+
+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.516609.patch
Type: text/x-patch
Size: 1522 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230425/26d14921/attachment.bin>