[cfe-commits] Evaluating a code review tool for clang/llvm: phabricator

[Sean Silva]

> Yes, this is specifically supported, and one of the reasons we're particularly interested in this toolset.

Very cool. I'll definitely be checking this out.

--Sean Silva

On Wed, Jul 11, 2012 at 2:59 PM, Chandler Carruth <chandlerc at google.com> wrote:
> On Wed, Jul 11, 2012 at 2:44 PM, Sean Silva <silvas at purdue.edu> wrote:
>>
>> Do you know if it is within phabricator's scope to notify you about
>> patches/commits that are in parts of the code that you're "interested
>> in watching"?
>
>
> Yes, this is specifically supported, and one of the reasons we're
> particularly interested in this toolset.
>
>>
>> This functionality is primarily to help developers
>> prioritize their code review time and make sure that they don't miss
>> patches relevant to them. Currently with the mailing lists, it is
>> difficult to filter patches/commits like this, and it seems like an
>> external tool is necessary.
>
>
> Exactly. It also supports automated "audits" and concepts such as code
> owners.
>
> And all of this works for post-commit as well as pre-commit.
>
> If the review process is suitable to the community, Manuel and I are
> interested in helping set up (and fix) these tools to closely match the
> existing community practices and expectations regarding post commit review
> and code ownership.