I sympathize with the concerns here; however, changing the default would be a very invasive change to Phabricator, which would not only be complex to implement but troublesome to maintain, as we upgrade Phabricator every week or two.
This is, however, something we can address with our new custom commit-series-friendly command-line tool. We are also working towards the superior solution of automatically selecting reviewers based on module owners and peers and enforcing this in Lando. Mark On Thu, Jul 5, 2018 at 11:37 AM, Andreas Tolfsen <a...@sny.no> wrote: > Also sprach Andrew Halberstadt: > > > It might be worth investigating whether we can switch Phabricator's > > default (so that multiple reviews are all blocking, and to make > > them non-blocking would require the extra step). > > I agree with Andrew. > > Making the default “everyone explicitly marked as reviewers must > accept the change” would in my opinion be much less surprising. > > It is then easier to use the UI to add other non-blocking reviewers > who might or might not be interested using the web UI. > _______________________________________________ > firefox-dev mailing list > firefox-...@mozilla.org > https://mail.mozilla.org/listinfo/firefox-dev > _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
