META: Code review flow which doesn't rely on manual assignment
Aim
It should only take one action to mark an MR as ready to review. It should then appear to eligible approvers on a Code Review page.
What
A page which shows non WIP MRs which I am eligible to review.
This page might also highlight MRs for which I am the only eligible approver so I know to approve them, as well as MRs which I have previously approved but where an additional commit has since removed that approval.
Why
When doing code review it is currently hard to tell which MRs need to be reviewed, especially when taking approval groups into account.
Because of this we end up with a workflow where we actively assign MRs to an individual for review. This results in having to find and select someone to review followed by a back and forth with users who may not be available to review at that time.
Example change to workflow for internal GitLab development
At the moment I write an MR, go to the GitLab projects handbook page, copy the list of reviewers into a text editor, go to google to pick a random number, go back to the text editor to check which line number that matches with, and then ask that person if they have time to review it, wait for their response and potentially reassign.
This proposal is that I instead just unmark my MR wip. I then go to my code review index and review someone else's MR. That page just needs to be able to show MRs which I have the ability to approve.