Use dangers to remind user to check migrations
What does this MR do and why?
Update danger so that it reminds folks to run the migration testing pipeline. Also calling to_set on the list of files because I can't quite understand why it's outputting duplicates. Also should fix files in db/docs not getting documentation review.
Related to #328367
How to set up and validate locally
- Comment out
danger/database/Dangerfile
Line 57return unless helper.ci?
- Generate a migration file with
rails g migration SomeRandomMigration
- Run
bin/rake danger_local
and see that the output says to kick off the migration testing pipeline.
MR 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 MR.
Edited by Alex Ives