Commit 76599631 authored by Andreas Brandl's avatar Andreas Brandl

Check migrations are reversible

parent a716292f
......@@ -104,6 +104,8 @@ and details for a database reviewer:
place them in `db/post_migrate` instead of `db/migrate`. Keep in mind
that post migrations are executed post-deployment in production.
- Check [timing guidelines for migrations](#timing-guidelines-for-migrations)
- Check migrations are reversible and implement a `#down` method
- Data migrations should be reversible, too or come with a description of how to reverse.
- Query performance
- Check for any obviously complex queries and queries the author specifically
points out for review (if any)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment