MR Reviewers should persist
One participant noted that when they accept edits from their reviewers, it cleared the approver(s) from the MR. This caused him to ping them again to re-accept the invite to be an approver on that MR as there were other changes to be made and approved. This increased the time needed to get a MR reviewed and approved.
Actionable Insight:
- Allow approvers to remain on a MR, even if their change was accepted, until the MR is merged for a final time.
Activity
marked this issue as related to ux-research#836 (closed)
added groupsource code label
Setting label(s) devopscreate sectiondev based on groupsource code.
added devopscreate sectiondev labels
-
Setting label(s) Category:Source Code Management based on groupsource code.
added Category:Source Code Management label
By default, approvals are reset on new pushes. There's a specific project setting for that. I'm going to close this as a “won't fix,” also because I wasn't able to find more issues about this.
@phikai FYI
added Category:Code Review Workflow approvals groupcode review merge requests labels and removed groupsource code label
removed Category:Source Code Management label
added Actionable InsightProduct change label
added closedwill not do label
