Closed
Bug 1531627
Opened 5 years ago
Closed 5 years ago
when updating a revision update reviewers from the local commit if the revision on phabricator doesn't have any reviewers
Categories
(Conduit :: moz-phab, enhancement, P3)
Conduit
moz-phab
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: glob, Assigned: zalun)
References
(Blocks 1 open bug)
Details
(Keywords: conduit-triaged)
Attachments
(1 file)
this is a subset of bug 1501077
This is one of my main annoyances with Phabricator/moz-phab as I often submit a WIP patch to a bug without reviewers and then later added reviewers when the patch is finished. This case has less issues related to conflicts as there are no reviewers on Phabricator… could we fix this more straightforward case in the short term and deal with the conflict cases later
Keywords: conduit-triaged
Summary: when updating a revision update reviewers from the local commit if the revision on phabricator doesn't have an reviewers → when updating a revision update reviewers from the local commit if the revision on phabricator doesn't have any reviewers
|
Assignee | |
Updated•5 years ago
|
Assignee: nobody → pzalewa
|
|
Assignee | |
Comment 3•5 years ago
|
||
|
||
Updated•5 years ago
|
Attachment #9053821 -
Attachment description: Bug 1531627 - WIP add reviewers on update → Bug 1501077 - WIP add reviewers on update
|
|
||
Updated•5 years ago
|
Attachment #9053821 -
Attachment description: Bug 1501077 - WIP add reviewers on update → Bug 1531627 - WIP add reviewers on update
|
|
||
Updated•5 years ago
|
Attachment #9053821 -
Attachment description: Bug 1531627 - WIP add reviewers on update → Bug 1531627 - Add reviewers on update r=glob
|
||
Comment 4•5 years ago
|
||
Should this bug have been closed? I think it was fixed.
Flags: needinfo?(pzalewa)
|
|
Assignee | |
Comment 5•5 years ago
|
||
just clearing the request. I will close it after the dependency will be fixed
Flags: needinfo?(pzalewa)
|
|
Assignee | |
Updated•5 years ago
|
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•