[llvm] 55f5a01 - Update Phabricator doc to remove the warning on "arc land": tags a properly handled server side now

Mehdi Amini via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 23 11:01:37 PDT 2020


Author: Mehdi Amini
Date: 2020-09-23T18:01:09Z
New Revision: 55f5a0137f5bbfc8411bd11a5d2695ba84534345

URL: https://github.com/llvm/llvm-project/commit/55f5a0137f5bbfc8411bd11a5d2695ba84534345
DIFF: https://github.com/llvm/llvm-project/commit/55f5a0137f5bbfc8411bd11a5d2695ba84534345.diff

LOG: Update Phabricator doc to remove the warning on "arc land": tags a properly handled server side now

Added: 
    

Modified: 
    llvm/docs/Phabricator.rst

Removed: 
    


################################################################################
diff  --git a/llvm/docs/Phabricator.rst b/llvm/docs/Phabricator.rst
index b1adb9be85d4..d6ec277939ee 100644
--- a/llvm/docs/Phabricator.rst
+++ b/llvm/docs/Phabricator.rst
@@ -182,14 +182,6 @@ that you close the review manually. In the web UI, under "Leap Into Action" put
 the git revision number in the Comment, set the Action to "Close Revision" and
 click Submit.  Note the review must have been Accepted first.
 
-Arcanist also adds extra tags that are mostly noise in the commit message, for
-this reason avoid using `arc land` and push commits to master directly with git
-after removing tags other than "Reviewed by" and "Differential Revision".
-You can run `llvm/utils/git/arcfilter.sh` to clean the commit message of the
-current "HEAD" commit automatically. You can also setup a git hook to catch this
-for you (see :ref:`git_pre_push_hook`).
-
-
 Committing someone's change from Phabricator
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 


        


More information about the llvm-commits mailing list