On Wed, Oct 17, 2012 at 8:25 PM, Sean Silva <span dir="ltr"><<a href="mailto:silvas@purdue.edu" target="_blank">silvas@purdue.edu</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
You might want to mention other functionality, like Herald, which I<br>
think fills a need that a lot of developers have.<br>
<br>
Well, I guess since this message is on the list, I can just say:<br>
Phabricator's "Herald" tool lets you (among other things) set up<br>
actions to happen in response to certain events; one huge use case for<br>
this is setting up alerts when parts of the tree that you are<br>
interested in watching get changed: this alleviates the problem of<br>
missing the chance to post-commit review on changes due to messages<br>
getting lost in the flood of llvm-commits mails.<br>
<br>
There's a lot of other functionality in Phabricator that I think might<br>
be useful; the big question is how to have it integrate properly into<br>
the mailing list---that's the reason I have held back in exploring<br>
some of the features.<br></blockquote><div><br></div><div>I'd say the biggest impact of a tool like Phabricator is for larger pre-commit code reviews (for example by answering the question "what has changed since the last patch"). Thus, that's the workflow we spent some time tuning. If we see that more people use and like Phabricator, we'll put more effort into making sure other features are integrated nicely with llvm's existing workflows.</div>
<div><br></div><div>Cheers,</div><div>/Manuel</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-- Sean Silva<br>
<div><div class="h5"><br>
On Wed, Oct 17, 2012 at 1:18 PM, Manuel Klimek <<a href="mailto:klimek@google.com">klimek@google.com</a>> wrote:<br>
> Dear LLVM / Clang community,<br>
><br>
> we'd like to open the use of Phabricator as an optional tool for doing code<br>
> reviews to a wider audience. Please feel free to start your code reviews by<br>
> following the documentation at <a href="http://llvm.org/docs/Phabricator.html" target="_blank">http://llvm.org/docs/Phabricator.html</a>.<br>
><br>
> Note that e-mail is still the reference medium for code reviews. Please let<br>
> me know about any problems with Phabricator or the documentation you find.<br>
><br>
> Cheers,<br>
> /Manuel<br>
><br>
</div></div>> _______________________________________________<br>
> cfe-dev mailing list<br>
> <a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
><br>
</blockquote></div><br></div>