Skip to content

Use participation/commenting within an MR as a factor for sort order

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

Proposal

Users who comment on merge requests are likely participants in the merge request. It's possible that they would be good choices for a review since they would be involved already and may be SMEs or CODEOWNERS for the changes being proposed.

We should weight people who comment more heavily within the list of users in the reviewer selector.

Original Description

Someone reviewed my MR recently and added some suggestions. I updated the code based on those suggestions, but the person had unassigned themselves as a Reviewer. I went back to assign them as a Reviewer again since I made those changes, but that person wasn't at the top of the Reviewers list, so I had to search their name and do it that way. It would be nice to populate the commenters of an MR at the top of the Reviewers dropdown so I don't have to search for them every time I want to re-assign them.

Screen_Shot_2021-08-30_at_9.37.52_AM

Additional Details

Commented By data is already available and used in the approvals widget so we might be able to use that as a potential new section or to help with sorting in the list.

Edited by 🤖 GitLab Bot 🤖