Page MenuHomePhabricator

remove automatic addition of "patch for review" if it can't also be automatically removed on merge
Closed, DuplicatePublic

Description

the automatic addion of the "patch for review" tag lacks the feature to also remove it again after a patch is merged and/or the ticket is closed.

that leads to either tons of tickets with that tag that don't actually have anything to review anymore, which means people won't be using it for the intended purpose,... or it means we have to manually remove it from every single task every single time we merge something

i would suggest to remove the automatic addition unless we can solve the issue how to remove it again on merge


See Also:
T95309: make gerritbot remove the "patch-for-review" tag once a patch is merged (or abandoned)
T61831: Tasks with abandoned changes shouldn't have Patch-for-Review

Event Timeline

Dzahn raised the priority of this task from to Needs Triage.
Dzahn updated the task description. (Show Details)
Dzahn added a project: Phabricator.
Dzahn subscribed.
mmodell triaged this task as Medium priority.Jul 7 2015, 2:43 PM
mmodell moved this task from To Triage to Need discussion on the Phabricator board.