diff options
Diffstat (limited to 'docs/Phabricator.rst')
-rw-r--r-- | docs/Phabricator.rst | 27 |
1 files changed, 20 insertions, 7 deletions
diff --git a/docs/Phabricator.rst b/docs/Phabricator.rst index 6fdea1f..581c9e5 100644 --- a/docs/Phabricator.rst +++ b/docs/Phabricator.rst @@ -12,13 +12,6 @@ you can now submit your patches for Clang and LLVM at Sign up ------- -There are two options to get an account on Phabricator. You can sign up -immediately with one of the supported OAuth account types if you're comfortable -with OAuth, but you can also email chandlerc@gmail.com to request an account to -be created manually without using OAuth. We're working to get support in -Phabricator to directly create new accounts, but currently this is a manual -process. - Note that if you use your Subversion user name as Phabricator user name, Phabricator will automatically connect your submits to your Phabricator user in the `Code Repository Browser`_. @@ -94,6 +87,26 @@ when a review changes state, for example by clicking "Accept Revision" in the web interface. Thus, please type LGTM into the comment box to accept a change from Phabricator. +Committing a change +------------------- + +When committing an LLVM change that has been reviewed using +Phabricator, the convention is for the commit message to end with the +line: + +:: + + Differential Revision: <URL> + +where ``<URL>`` is the URL for the code review, starting with +``http://llvm-reviews.chandlerc.com/``. + +Note that Arcanist will add this automatically. + +This allows people reading the version history to see the review for +context. This also allows Phabricator to detect the commit, close the +review, and add a link from the review to the commit. + Status ------ |