[llvm] r208764 - Update of the documentation: I think we are now happy with Phabricator
tobias at grosser.es
Wed May 14 02:37:41 PDT 2014
On 14/05/2014 11:22, Sylvestre Ledru wrote:
> Author: sylvestre
> Date: Wed May 14 04:22:15 2014
> New Revision: 208764
> URL: http://llvm.org/viewvc/llvm-project?rev=208764&view=rev
> Update of the documentation: I think we are now happy with Phabricator
> Modified: llvm/trunk/docs/Phabricator.rst
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/docs/Phabricator.rst?rev=208764&r1=208763&r2=208764&view=diff
> --- llvm/trunk/docs/Phabricator.rst (original)
> +++ llvm/trunk/docs/Phabricator.rst Wed May 14 04:22:15 2014
> @@ -110,8 +110,7 @@ review, and add a link from the review t
> -Currently, we're testing Phabricator for use with Clang/LLVM. Please let us
> -know whether you like it and what could be improved!
> +Please let us know whether you like it and what could be improved!
We still don't get email reviews in phabricator. This patch was e.g.
reviewed by Nick, but the review does not show up in phabricator.
In general, the support for email reviews is lacking, which makes
offline patch reviews difficult to use e.g. when travelling/commuting.
More information about the llvm-commits