[llvm] r175047 - Update phab docs to clarify how to accept a change.

Manuel Klimek klimek at google.com
Wed Feb 13 01:07:19 PST 2013


Author: klimek
Date: Wed Feb 13 03:07:18 2013
New Revision: 175047

URL: http://llvm.org/viewvc/llvm-project?rev=175047&view=rev
Log:
Update phab docs to clarify how to accept a change.

Modified:
    llvm/trunk/docs/Phabricator.rst

Modified: llvm/trunk/docs/Phabricator.rst
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/docs/Phabricator.rst?rev=175047&r1=175046&r2=175047&view=diff
==============================================================================
--- llvm/trunk/docs/Phabricator.rst (original)
+++ llvm/trunk/docs/Phabricator.rst Wed Feb 13 03:07:18 2013
@@ -88,6 +88,12 @@ diffs between different versions of the
 *Revision Update History*. Most features are self descriptive - explore, and
 if you have a question, drop by on #llvm in IRC to get help.
 
+Note that as e-mail is the system of reference for code reviews, and some
+people prefer it over a web interface, we do not generate automated mail
+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.
+
 Status
 ------
 





More information about the llvm-commits mailing list