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

Sean Silva silvas at purdue.edu
Wed Jul 11 14:44:11 PDT 2012


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"? 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.

This "filtering" originally came up in a discussion with Michael
Spencer; he might be able to more clearly enunciate the functionality
he was envisioning.

--Sean Silva

On Wed, Jul 11, 2012 at 10:33 AM, Manuel Klimek <klimek at google.com> wrote:
> Hi,
>
> as you might have noticed by now due to emails from
> reviews at llvm-reviews.chandlerc.com we're in the process of evaluating
> phabricator (http://phabricator.org) to support clang/llvm code
> reviews. To find out how well phabricator fits clang/llvm's code
> review process, we're starting to do code reviews with it and file
> feature requests against phabricator (mainly around email handling).
>
> If you have more questions about phabricator feel free to reach out to
> me or Chandler, or join #phabricator on freenode to chat with the
> phabricator devs.
>
> Also feel free to play around with phabricator at
> http://llvm-reviews.chandlerc.com. We're still experimenting, so
> please be aware that this is not an official way to do code reviews.
>
> Happy hacking,
> /Manuel
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits



More information about the cfe-commits mailing list