Skip to content

Removes maintainer only reviewers

Vijay Hawoldar requested to merge vij-maintainer-only-reviews into master

What does this MR do and why?

In https://gitlab.com/gitlab-org/gitlab-roulette/-/merge_requests/84 we added support for reviewers to indicate that they only wish to have maintainer reviewers, using Gitlab/Slack status emoji (Ⓜ).

This MR attempt to use that new attribute to determine if we should remove eligible reviewers from suggestions.

The reason for supporting this is to enable maintainers to be more available for reviews on projects they are maintainers on, rather than limiting their capacity which might still be filled with non-maintainer reviews on other projects. More details/discussion can be found in https://gitlab.com/gitlab-org/gitlab-roulette/-/merge_requests/84 and https://gitlab.com/gitlab-org/fulfillment-meta/-/issues/874

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Vijay Hawoldar

Merge request reports