Please add Herald group and rule for new "browser-installer-reviewers" group
Categories
(Conduit :: Phabricator, defect)
Tracking
(Not tracked)
People
(Reporter: nalexander, Assigned: dkl)
References
Details
Could we please add browser/installer
to the regex that controls when the application-update-reviewers
group is added to a patch? Thanks!
Reporter | ||
Comment 1•1 month ago
|
||
Actually, could we make this a new browser-installer-reviewers
group, with a herald rule for browser/installer
; and could we add me (:nalexander), Chris DuPuis (:cdupuis), and Nicholas Rishel (:nrishel) as members? Thanks!
Assignee | ||
Updated•1 month ago
|
Assignee | ||
Comment 2•1 month ago
|
||
Done
https://phabricator.services.mozilla.com/project/profile/221
https://phabricator.services.mozilla.com/H474
Reporter | ||
Comment 3•1 month ago
|
||
(In reply to David Lawrence [:dkl] from comment #2)
Done
https://phabricator.services.mozilla.com/project/profile/221
https://phabricator.services.mozilla.com/H474
dkl: could I trouble you to make that browser/installer/windows
? I was wide of the mark. If you want the name of the group to include Windows as well, feel free to make it #windows-installer-reviewers
or similar.
Assignee | ||
Comment 4•1 month ago
|
||
(In reply to Nick Alexander :nalexander [he/him] from comment #3)
(In reply to David Lawrence [:dkl] from comment #2)
Done
https://phabricator.services.mozilla.com/project/profile/221
https://phabricator.services.mozilla.com/H474dkl: could I trouble you to make that
browser/installer/windows
? I was wide of the mark. If you want the name of the group to include Windows as well, feel free to make it#windows-installer-reviewers
or similar.
Description
•