Roulette should suggest QA maintainer when author is SET.
Roulette should suggest QA maintainer when author is SET.
From slack thread: https://gitlab.slack.com/archives/CMA7DQJRX/p1611174005026100
@a_mcdonald: Hi team, I noticed that all recent MRs don't have Danger recommendations for a QA maintainer - was it an intentional change? I can imagine it is optional for frontend work but seems odd for E2E MRs like !51870 (merged) Just thought I'll check in case it's a bug
@caalberts: THere’s this b1c6535f
@a_mcdonald: Hmm, then it doesn't work as expected because the author is a SET and maintainer should be mandatory according to this rule?
@caalberts: Maybe it’s this line: b1c6535f It assumes that SET is a reviewer qa, but I notice that for you, it’s: projects: gitlab: trainee_maintainer qa gitlab-qa: reviewer