Experimenting with Phabricator for reviews

Jun Wu quark at fb.com
Fri Jul 14 18:54:44 EDT 2017


Excerpts from Augie Fackler's message of 2017-07-14 12:09:10 -0400:
> If there's a way to get just "new review" messages from phabricator
> into the list, let's do that. I _do not_ want us to stop notifying the
> list of phabricator reviews, because then people that don't watch
> phabricator have no visibility into changes happening there, and we
> split our community.

My feeling is that both people who care about, and who don't care about
Phabricator will want to send those emails to /dev/null. Reasons:

  - For people not using Phabricator, they surely don't care about the
    notifications and want to nuke them.
  - For people active on Phabricator, they read the notification there and
    don't need additional email notifications. (I belong to this category
    and I send those emails to /dev/null already).

So it seems to me that the only value of getting those emails is the
"achieving" purpose. One brainstorm idea from an internal discussion is to
send the emails to a separated list. It seems the clang community is doing
something similar - they have "-dev" list and "-commits" list. I guess
nobody really reads the "-commits" list and that seems for achieving only.

> (So far I'm not at all happy with Phabricator and how it's split the
> review load in an awkward way - for this reviewer, it's been a clear
> negative.)


More information about the Mercurial-devel mailing list