[Reviewers] Recommending Phabricator

Augie Fackler raf at durin42.com
Thu Feb 8 14:19:06 UTC 2018


(+Ryan since he's got a contact at Phacility)

> On Feb 8, 2018, at 13:03, Yuya Nishihara <yuya at tcha.org> wrote:
> 
> On Wed, 7 Feb 2018 09:54:39 -0800, Gregory Szorc wrote:
>> One of the changes I made was to recommend Phabricator over emailing
>> patches. I did this because I think Phabricator is more friendly to new
>> contributors and provides a gateway for new contributors getting more
>> involved.
> 
> I hope not advertising Phabricator more until its email interface gets fixed.
> Proper threading and inline-reply are must IMO.
> 
>> In addition, I think the time has come for more of Mercurial's core
>> contributors to transition from email to Phabricator.
> 
> I thought that was a temporary issue of email infrastructure at Google?
> 
>> As a reviewer, I've noticed I'm biasing to looking at Phabricator first
>> then falling back to Patchwork if I still want to do reviews. Phabricator
>> is just a more pleasant experience for me as both a reviewer and a
>> contributor.
> 
> I do truly the opposite. Ignoring almost all Phabricator emails, and pick
> random patches, which I would have to review, from Yadda using phabread.

I still check both. I'm quicker at email reviews and find them preferable, but I've also made peace with the seeming reality that new contributors are significantly more comfortable with phabricator.

I would like it if we could try and get some more improvements figured out, but IIRC the email threading thing was a complete nonstarter for them? Greg, do you know if Mozilla is likely to care about that?

> _______________________________________________
> Reviewers mailing list
> Reviewers at mercurial-scm.org
> https://www.mercurial-scm.org/mailman/listinfo/reviewers




More information about the Reviewers mailing list