lfittl / activerecord-clean-db-structure

Automatic cleanup for the Rails db/structure.sql file (ActiveRecord/PostgreSQL)
BSD 3-Clause "New" or "Revised" License
146 stars 36 forks source link

Organize schema_migrations values to prevent merge conflicts #15

Closed nvanoorschot closed 5 years ago

nvanoorschot commented 5 years ago

fixes #14

lfittl commented 5 years ago

@nvanoorschot I think this is a good idea!

However, to avoid surprises for existing users of this gem, how do you feel about introducing this as an optional method first? See https://github.com/lfittl/activerecord-clean-db-structure/pull/13 for an example of such an option.

nvanoorschot commented 5 years ago

@lfittl Good idea, I turned it into an option, keeping the output the same for those not opting in.

lfittl commented 5 years ago

@nvanoorschot Thanks for adjusting - sorry for the radio silence. Merging now :)