maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

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

This commit is contained in:
Mehdi Amini 2020-09-23 17:57:27 +00:00
parent b901b6ab17
commit 55f5a0137f
1 changed files with 0 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
llvm/docs/Phabricator.rst
View File

@ -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 the git revision number in the Comment, set the Action to "Close Revision" and
click Submit. Note the review must have been Accepted first. 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 Committing someone's change from Phabricator
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^