<div dir="ltr"><div>It might be worth investigating whether we can switch Phabricator's</div><div>default (so that multiple reviews are all blocking, and to make them</div><div>non-blocking would require the extra step). Personally I think setting</div><div>multiple reviewers up on a first come first serve is disrespectful to</div><div>those reviewers' time, so shouldn't be the case that gets optimized.</div><div><br></div><div>Though I also like that the distinction can be made, and there are cases</div><div>where first-come-first-serve would be a good idea (e.g things that need</div><div>to land ASAP).</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jul 3, 2018 at 1:29 AM glob <<a href="mailto:glob@mozilla.com">glob@mozilla.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div style="font-family:tt" text="#000000" bgcolor="#FFFFFF"><div style="font-family:tt"><span>Jean-Yves Avenard wrote on 3/7/18 6:23 
am:</span></div></div><div style="font-family:tt" text="#000000" bgcolor="#FFFFFF"><div style="font-family:tt"><br><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jul
 2, 2018 at 5:01 PM, 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 Marco 
Bonardo:<br>
<span><br>
> When asking for review to multiple reviewers, and all of them must 
accept<br>
> your revision, you must mark them as blocking reviews, either in 
the<br>
> Phabricator ui or appending "!" at the end of the reviewer name. 
Otherwise<br>
> it's first-come-first-serve.<br>
<br>
</span>Note that is and also has been the case for mozreview.<br></blockquote></div><div class="gmail_quote"><br></div><div class="gmail_quote">I don't ever 
recall mozreview having different kind of reviewer (blocker or 
non-blocker), if two people were added as reviewer, by default both had 
to review.</div></div></div></blockquote></div></div><div style="font-family:tt" text="#000000" bgcolor="#FFFFFF"><div style="font-family:tt">it's correct that mozreview 
(and bugzilla) only have one type of reviewer.  what multiple reviewers 
means in bugzilla/mozreview varies from team to team (all must review 
vs. any can review).<br><br>it isn't correct that in mozreview two 
reviewers would both have to review.<br>approval from _any_ reviewer 
would allow it to be landed with autoland:<br><a class="m_-636699594234998274moz-txt-link-freetext" href="https://hg.mozilla.org/hgcustom/version-control-tools/file/tip/pylib/mozreview/mozreview/review_helpers.py#l34" target="_blank">https://hg.mozilla.org/hgcustom/version-control-tools/file/tip/pylib/mozreview/mozreview/review_helpers.py#l34</a><br><br>i
 like that phabricator makes this distinction up-front.<br>thanks mak 
for drawing attention to this difference/feature.</div></div><div style="font-family:tt" text="#000000" bgcolor="#FFFFFF"><div style="font-family:tt"><br><br><br>-glob<br><div class="m_-636699594234998274moz-signature">-- <br><span style="color:rgb(192,192,192)">glob
 — engineering workflow — moz://a</span><br>



<br>

</div></div></div>
_______________________________________________<br>
firefox-dev mailing list<br>
<a href="mailto:firefox-dev@mozilla.org" target="_blank">firefox-dev@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/firefox-dev" rel="noreferrer" target="_blank">https://mail.mozilla.org/listinfo/firefox-dev</a><br>
</blockquote></div>