Skip to content

Add ability to include and exclude CODEOWNERS rules for roulette

Nick Nguyen requested to merge nhxnguyen-configurable-codeowners into master

What does this MR do and why?

This MR adds the ability to configure reviewer roulette with the following options when it comes to CODEOWNERS approvals:

  • Include specified optional CODEOWNERS sections via config.included_optional_codeowners_sections_for_roulette
  • Exclude specified required CODEOWNERS sections via config.excluded_required_codeowners_sections_for_roulette

The reasons to provide these options include, but are not limited to:

  • Allowing us to recommend roulette reviewers based on CODEOWNERS without making the relevant sections required.
  • Allowing us to require CODEOWNERS sections that already have existing roulette support without adding a duplicate suggestion.

For more context, see #45 (closed)

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 Nick Nguyen

Merge request reports

Loading