752
edits
Phabricator/ReviewerRotation: Difference between revisions
Jump to navigation
Jump to search
no edit summary
DaveLawrence (talk | contribs) No edit summary |
DaveLawrence (talk | contribs) No edit summary |
||
| Line 5: | Line 5: | ||
'''Setting up a reviewer rotation group:''' | '''Setting up a reviewer rotation group:''' | ||
* A bug is filed in Bugzilla in the Conduit product requesting that a new reviewer rotation group should be added and the initial members of the group. A Herald rule an also be requested to automatically add the rotation group to certain new revisions that match the criteria. | * A bug is filed in Bugzilla in the [https://bugzilla.mozilla.org/enter_bug.cgi?product=Conduit&component=Phabricator Conduit product] requesting that a new reviewer rotation group should be added and the initial members of the group. A Herald rule an also be requested to automatically add the rotation group to certain new revisions that match the criteria. | ||
* A new reviewer rotation group is created in Phabricator and named such it ends in -rotation. Example: toolkit-telemetry-reviewers-rotation. The -reviewers-rotation part lets PhabBugz determine which groups to manage. | * A new reviewer rotation group is created in Phabricator and named such it ends in -rotation. Example: toolkit-telemetry-reviewers-rotation. The -reviewers-rotation part lets PhabBugz determine which groups to manage. | ||
* Herald rules will be set up to add the reviewer rotation group automatically when new revisions are created. Normally the Herald rule will look at file paths, etc. to determine when to add the group. | * Herald rules will be set up to add the reviewer rotation group automatically when new revisions are created. Normally the Herald rule will look at file paths, etc. to determine when to add the group. | ||