]> git.openstreetmap.org Git - rails.git/commit
Disable Rails/BulkChangeTable cop
authorAndy Allan <git@gravitystorm.co.uk>
Wed, 19 Jun 2024 16:44:20 +0000 (17:44 +0100)
committerAndy Allan <git@gravitystorm.co.uk>
Wed, 19 Jun 2024 16:44:20 +0000 (17:44 +0100)
commitf1fc31d485eb310781600645df43b3f45a76fd57
treec0ffcd38cef15bd3cdeb0087b28b17cf0290f28d
parentd57dd1f4da4d4605f534607b69944e3c977e4db5
Disable Rails/BulkChangeTable cop

If you have multiple alter commands in a migration (like adding two
columns to an existing table) this cop will suggest combining them using
`change_table :foo, :bulk => true`.

However, Strong Migrations can't inspect `change_table` blocks, and
therefore suggests using `safety_assured`, which mostly defeats the
purpose of using strong_migrations in the first place.

Instead, we should stick with the individual alter commands. Since
postgres runs DDL changes in a transaction anyway, there is not enough
benefit from the `change_table` approach to be worth losing the
safety net of the Strong Migrations checks.
.rubocop.yml