From 51e31d211cfd868623b849c3f3d6c4efc95a5303 Mon Sep 17 00:00:00 2001 From: Manuel Klimek Date: Wed, 13 Feb 2013 09:07:18 +0000 Subject: [PATCH] Update phab docs to clarify how to accept a change. llvm-svn: 175047 --- llvm/docs/Phabricator.rst | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/llvm/docs/Phabricator.rst b/llvm/docs/Phabricator.rst index b45449793e0a..efab10cd13a5 100644 --- a/llvm/docs/Phabricator.rst +++ b/llvm/docs/Phabricator.rst @@ -88,6 +88,12 @@ diffs between different versions of the patch as it was reviewed in 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 ------