philbooth · GitHub

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tbh, I see the value of these lints not in checking migrations that we already wrote, but in checking migrations we're yet to write.

If we're really interested in why something fails right now, we can just remove the entry here and run the script. It will tell us, but that knowledge is academic because we can't go back and change the old migration anyway. But when write a new migration, it will tell us what we've done wrong at that point, which is where the real value is. Hopefully that will stop us deploying problematic migrations to prod in the future.

Read the original on github.com ↗