<div dir="ltr"><div>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.</div><div><br></div><div>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.<br></div><div><br></div><div>Mark</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 5, 2018 at 11:37 AM, Andreas Tolfsen <span dir="ltr"><<a href="mailto:ato@sny.no" target="_blank">ato@sny.no</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Also sprach Andrew Halberstadt:<br>
<span class=""><br>
> It might be worth investigating whether we can switch Phabricator's<br>
> default (so that multiple reviews are all blocking, and to make<br>
> them non-blocking would require the extra step).<br>
<br>
</span>I agree with Andrew.<br>
<br>
Making the default “everyone explicitly marked as reviewers must<br>
accept the change” would in my opinion be much less surprising.<br>
<br>
It is then easier to use the UI to add other non-blocking reviewers<br>
who might or might not be interested using the web UI.<br>
<div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
firefox-dev mailing list<br>
<a href="mailto:firefox-dev@mozilla.org">firefox-dev@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/firefox-dev" rel="noreferrer" target="_blank">https://mail.mozilla.org/<wbr>listinfo/firefox-dev</a><br>
</div></div></blockquote></div><br></div>