Skip to content

Consider preventing editing reviewers after a merge request has been merged

Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.

Proposal

Prevent reviewers from being edited after a merge request has been merged.

At the moment, when a merge request is merged the actions to Assign reviewers, Assign yourself, and Edit for the reviewers section in the sidebar are still editable. If there is no need to amend post merge, then we should be able to safely hide those actions on the page.

image

Questions

  • What is the scenario of needing to ammend reviewers post merge?
  • What are potential workarounds?
    • In the case that someone wants to amend then they would re-open the MR, amend it, then close it again? Wouldn't that change the MR to be closed rather than merged? Need to verify.
Edited by 🤖 GitLab Bot 🤖