Experimenting with Phabricator for reviews

[Sean Farley]

Sean Farley <sean at farley.io> writes:

> Adrian Buehlmann <adrian at cadifra.com> writes:
>
>> On 2017-07-10 22:00, Kevin Bullock 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.
>>
>> Ugh. Looks way too chatty/cluttered for my taste so far (I'm mostly
>> lurking the list).
>>
>> I'm likely going to unsubscribe Mercurial-devel, if it stays like that...
>
> Yeah, I've found this to be pretty ugly as well; though I don't think
> it's a function of chatty-ness but rather lack of threading :-(

Another thing that I'd like: can we drop the "[Differential] [Foo
blah]"? It seems mostly useless information and wastes precious
horizontal real estate.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 800 bytes
Desc: not available
URL: <http://www.mercurial-scm.org/pipermail/mercurial-devel/attachments/20170712/fafaadd8/attachment.sig>