Change AR::ConnectionAdapters::SchemaStatements#add_index to take kwargs


I’m always confused by the existence of uniq and unique. Arrays and ActiveRecord::Relation objects respond to uniq while AR::ConnectionAdapters::SchemaStatements#add_index takes :unique option. I would like to get notified when :uniq option is actually wrong. If we change #add_index to take kwargs, it’ll raise an error when :uniq option is given.

Does this make sense? Thanks for your time again!



Never mind, I guess the train I was on last week was way too old: