[llvm] r208764 - Update of the documentation: I think we are now happy with Phabricator

[Tobias Grosser]

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
> Log:
> Update of the documentation: I think we are now happy with Phabricator
>
> 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=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
>   Status
>   ------
>
> -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.

http://reviews.llvm.org/D3754

In general, the support for email reviews is lacking, which makes 
offline patch reviews difficult to use e.g. when travelling/commuting.

Cheers,
Tobias