[PATCH] D110873: [docs] Expand the pre-merge testing description a bit

[Philip Reames via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rGe2f150c3cf8b: [docs] Expand the pre-merge testing description a bit (authored by reames).

Changed prior to commit:
  https://reviews.llvm.org/D110873?vs=376320&id=377412#toc

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110873

Files:
  llvm/docs/Phabricator.rst


Index: llvm/docs/Phabricator.rst
===================================================================
--- llvm/docs/Phabricator.rst
+++ llvm/docs/Phabricator.rst
@@ -181,7 +181,12 @@
 
 When a user uploads a patch to Phabricator, Phabricator triggers the checks and
 then displays the results. This way bugs in a patch are contained during the 
-code review stage and do not pollute the main branch. 
+code review stage and do not pollute the main branch.
+
+Our goal with pre-merge testing is to report most true problems while strongly
+minimizing the number of false positive reports.  Our goal is that problems
+reported are always actionable.  If you notice a false positive, please report
+it so that we can identify the cause.
 
 If you notice issues or have an idea on how to improve pre-merge checks, please 
 `create a new issue <https://github.com/google/llvm-premerge-checks/issues/new>`_ 
@@ -217,6 +222,25 @@
 
   .. image:: Phabricator_premerge_unit_tests.png
 
+Opting Out
+^^^^^^^^^^
+
+In case you want to opt-out entirely of pre-merge testing, add yourself to the
+`OPT OUT project <https://reviews.llvm.org/project/view/83/>`_.  If you decide
+to opt-out, please let us know why, so we might be able to improve in the future.
+
+Operational Details
+^^^^^^^^^^^^^^^^^^^
+
+The code responsible for running the pre-merge flow can be found in the external
+repository  `https://github.com/google/llvm-premerge-checks`_.  For enhancement
+ideas and most bugs, please file an issue on said repository.  For immediate
+operational problems, the point of contact is
+`Mikhail Goncharov <mailto:goncharo at google.com>`_.
+
+Background on the pre-merge infrastructure can be found in `this 2020 DevMeeting
+talk <https://llvm.org/devmtg/2020-09/slides/Goncharov-Pre-merge_checks.pdf>`_
+
 Committing a change
 -------------------
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D110873.377412.patch
Type: text/x-patch
Size: 1841 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211006/f2b7daae/attachment.bin>