From 55f5a0137f5bbfc8411bd11a5d2695ba84534345 Mon Sep 17 00:00:00 2001 From: Mehdi Amini Date: Wed, 23 Sep 2020 17:57:27 +0000 Subject: [PATCH] Update Phabricator doc to remove the warning on "arc land": tags a properly handled server side now --- llvm/docs/Phabricator.rst | 8 -------- 1 file changed, 8 deletions(-) diff --git a/llvm/docs/Phabricator.rst b/llvm/docs/Phabricator.rst index b1adb9b..d6ec277 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 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ -- 2.7.4