Skip to content

Draft: Include maintainers in the reviewers pool

Paul Gascou-Vaillancourt requested to merge maintainers-as-reviewers into master

What does this MR do and why?

This ensures that maintainers are part of the initial reviewers pool. There's a general understanding that maintainers should be pickable for initial reviews, but this isn't reflected in the Danger roulette.

The intent of this change is to reduce the burden of initial reviewers in the frontend department where there are currently more maintainer than reviewers, and the latter are generally less available than the former.

More context: https://gitlab.slack.com/archives/C9Q5V0597/p1678395377302829 (internal).

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 Paul Gascou-Vaillancourt

Merge request reports