[Proposal]: Make all reference constraints deferrable about strong_migrations HOT 4 CLOSED

Hey @volkanunsal, thanks for the proposal 👍 Can you add an example of a migration that should raise an error, and as well as one that does it properly?

from strong_migrations.

I don't quite know how to express the correct way of doing it. The way to alter a constraint requires knowing the name of the constraint, but that name is generated by Rails, and not easily accessible to the programmer.

Here is an attempt to show what I mean:

wrong

class AddReferenceToProject < ActiveRecord::Migration[5.1]
  def change
    add_foreign_key :users, :projects
  end
end

right

class AddReferenceToProject < ActiveRecord::Migration[5.1]
  def change
    # Naming our own foreign key. Is there a better way?
    add_foreign_key :users, :projects, name: 'foobar'
    execute("ALTER TABLE users ALTER CONSTRAINT foobar DEFERRABLE INITIALLY IMMEDIATE;")
  end
end

from strong_migrations.

Hey @volkanunsal, thanks for following up. I don't know if this practice is common, so don't think it makes sense to add right now. I'd like to add support for custom rules at some point, which could allow you to create your own warning around this.

from strong_migrations.

fwiw, just added a way to write custom checks

from strong_migrations.