Confirmed users
1,201
edits
| Line 28: | Line 28: | ||
=== How do I '''require''' a review from all reviewers before landing? === | === How do I '''require''' a review from all reviewers before landing? === | ||
Add all reviewers as "blocking" reviewers, either from the UI or by appending a “!” to their name when specifying them in Arcanist | Add all reviewers as "blocking" reviewers, either from the UI or by appending a “!” to their name when specifying them in MozPhab or Arcanist: | ||
<code>Bug 123456 - Example commit description; r=someone!</code> | |||
With MozPhab you can alternatively use the <code>--blocker</code> switch: | |||
<code>moz-phab submit --blocker someone</code> | |||
To set a reviewer as blocking in the UI, edit the revision and use the reviewer autocomplete; each result will have a normal and a "blocking" entry. | |||
[[File:blocking_reviewers.png|640px|Selecting blocking vs nonblocking reviewers]] | [[File:blocking_reviewers.png|640px|Selecting blocking vs nonblocking reviewers]] | ||