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.
-
I have evaluated the MR acceptance checklist for this merge request.
Edited by Evan Read