Skip to content

Post message in MRs instead of using simple_roulette

What does this MR do and why?

The gitlab-docs project currently uses simple_roulette (https://gitlab.com/gitlab-org/ruby/gems/gitlab-dangerfiles/-/blob/master/lib/danger/rules/simple_roulette/Dangerfile), the results of which can be seen in merge requests such as:

It's rarely what you want, and I assume is ignored almost 100% of the time.

More automation will no doubt be possible later (detect the files that have changed and suggest based on that), but an easy win is just to post a simple message to the MR (like the one on this MR) stating our expectations for who should review what.

Merge request 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 Evan Read

Merge request reports