Skip to content

Reviewer roulette as part of the danger rules

We currently have a basic, informal chatops integration for "reviewer roulette": https://gitlab.com/dennis/reviewer-roulette . However, it doesn't satisfy all our needs, and slack isn't obviously the best place for the interface to this functionality. I think the integration is actually broken at present as well.

In discussion with @DouweM the idea of subsuming the "roulette" function into our existing danger bot came up. It already suggests appropriate reviewers for the documentation team when an MR touches the doc/ hierarchy; I think we could have it suggest appropriate reviewers and maintainers for an MR generally, including a roulette element, while taking information like OOO status, CODEOWNERS, and "expert in..." into account.

cc @gitlab-org/maintainers/rails-backend