spec/rubocop/cop · f87f4250dda17e2f0e95d65b7a23255f665c1cfc · Forest Godfrey / gitlab-ce

Add cop for use of remove_column · 1ab33b15

authored Dec 11, 2017

remove_column should only be used in the up (or change) step of a migration if
it's a post-deployment migration. Otherwise there will be downtime due to the
ActiveRecord column cache, which we can avoid by using the IgnorableColumn
concern in combination with a post-deployment migration.

1ab33b15

Name	Last commit	Last update
..
migration		Loading commit data...
rspec		Loading commit data...
active_record_dependent_spec.rb		Loading commit data...
active_record_serialize_spec.rb		Loading commit data...
custom_error_class_spec.rb		Loading commit data...
gem_fetcher_spec.rb		Loading commit data...
in_batches_spec.rb		Loading commit data...
line_break_after_guard_clauses_spec.rb		Loading commit data...
polymorphic_associations_spec.rb		Loading commit data...
project_path_helper_spec.rb		Loading commit data...
redirect_with_status_spec.rb		Loading commit data...