Experimenting with Phabricator for reviews

Gregory Szorc gregory.szorc at gmail.com
Tue Jul 11 13:36:50 EDT 2017


On Mon, Jul 10, 2017 at 1:00 PM, Kevin Bullock <
kbullock+mercurial at ringworld.org> wrote:

> Greetings, Mercurial hackers!
>
> We've set up an instance of Phabricator that we're going to experiment
> with for code reviews. At this time, we're not planning on using any of the
> other features of Phabricator, and the use of Phabricator for reviews is
> very much *an experiment* - we intend to see how this is going in a little
> while (probably around the start of the 4.4 cycle). If you're more
> comfortable submitting and reviewing patches as plain email, that's fine -
> use of Phabricator is optional. In the meantime, you'll see e-mails coming
> to the mailing list when activity happens in Phabricator--hopefully we've
> hit the right settings to make this useful without being too chatty.
>
> See https://www.mercurial-scm.org/wiki/Phabricator for a little more
> detail, including how to submit new reviews.
>

\o/

Regarding the email to this list, is it a feature or a bug that review
activity at https://phab.mercurial-scm.org/D31 and D32 isn't making it to
this list? I would expect at least a "review created" email to make its way
to the list. (Although it isn't clear the Harold H1 rule for notifying this
list was in place when that review was created.)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.mercurial-scm.org/pipermail/mercurial-devel/attachments/20170711/ebe22597/attachment.html>


More information about the Mercurial-devel mailing list