[llvm] Document use of `skip-precommit-approval` label for non-review pull requests (PR #81053)

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 7 15:27:30 PST 2024


https://github.com/dwblaikie created https://github.com/llvm/llvm-project/pull/81053

Derived from this discussion: https://discourse.llvm.org/t/prs-without-approvals-muddy-the-waters/76656


>From a923d34a6e2c2196a5a0ebca13f8bae171c05aaf Mon Sep 17 00:00:00 2001
From: David Blaikie <dblaikie at gmail.com>
Date: Wed, 7 Feb 2024 03:13:37 +0000
Subject: [PATCH] Document use of `skip-precommit-approval` label for
 non-review pull requests

Derived from this discussion: https://discourse.llvm.org/t/prs-without-approvals-muddy-the-waters/76656
---
 llvm/docs/CodeReview.rst | 6 ++++++
 llvm/docs/GitHub.rst     | 6 ++++++
 2 files changed, 12 insertions(+)

diff --git a/llvm/docs/CodeReview.rst b/llvm/docs/CodeReview.rst
index f1d5b6c9c99206..e1225ee01ac2f8 100644
--- a/llvm/docs/CodeReview.rst
+++ b/llvm/docs/CodeReview.rst
@@ -103,6 +103,12 @@ ready to be committed. Specifically, once a patch is sent out for review, it
 needs an explicit approval before it is committed. Do not assume silent
 approval, or solicit objections to a patch with a deadline.
 
+.. note::
+   If you are using a Pull Request to get precommit CI results, but not
+   because it needs a precommit review, apply the `skip-precommit-approval
+   <https://github.com/llvm/llvm-project/labels?q=skip-precommit-approval>`
+   label to the PR.
+
 Acknowledge All Reviewer Feedback
 ---------------------------------
 
diff --git a/llvm/docs/GitHub.rst b/llvm/docs/GitHub.rst
index c3fbb64c2dd96a..91d431012c8bfa 100644
--- a/llvm/docs/GitHub.rst
+++ b/llvm/docs/GitHub.rst
@@ -30,6 +30,12 @@ describes the typical workflow of creating a Pull Request and getting it reviewe
 and accepted. This is meant as an overview of the GitHub workflow, for complete
 documentation refer to `GitHub's documentation <https://docs.github.com/pull-requests>`_.
 
+.. note::
+   If you are using a Pull Request to get precommit CI results, but not
+   because it needs a precommit review, apply the `skip-precommit-approval
+   <https://github.com/llvm/llvm-project/labels?q=skip-precommit-approval>`
+   label to the PR.
+
 GitHub Tools
 ------------
 You can interact with GitHub in several ways: via git command line tools,



More information about the llvm-commits mailing list