Skip to content

Add danger error comment when altering sidekiq arguments

Max Woolf requested to merge mw/sidekiq-arg-commenter into master

What does this MR do and why?

Adds a danger rule to warn MR authors when adding a new sidekiq argument that there is a process to follow that covers multiple milestones.

This was a corrective action after a recent feature flag rollout caused a production incident.

Screenshots or screen recordings

Screenshot_2022-05-13_at_15.37.03

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Mentions #362192 (closed)

Edited by Max Woolf

Merge request reports