Skip to content

Add substitutes for maintainers in Danger roulette

Sean Arnold requested to merge add-substitue-options-for-danger-roulette into master

What does this MR do?

Related issue: !18711 (merged)

Due to the extreme shortage (internal) of DB maintainers, we've been assigning Reviewers and Traintainers to provide a second review in lieu of a Maintainer being available.

This adds a substitutes option to easily pass in as a backup list of people to choose when the original list of people doesn't have a matching result.

Open questions:

  • What people group do we want to use in lieu of Maintainers? I've currently chosen Traintainers but this is being discussed here: !18711 (diffs, comment 231708356)
  • Do we want to write some MARKDOWN when we use the substitute list?

Conformity

Availability and Testing

Tests have been added but we'll need test before going live, probably.

Edited by Sean Arnold

Merge request reports