gitlab-org--gitlab-foss/db/migrate/20140414131055_change_state...

# rubocop:disable all
class ChangeStateToAllowEmptyMergeRequestDiffs < ActiveRecord::Migration[4.2]
  def up
    change_column :merge_request_diffs, :state, :string, null: true,
                  default: nil
  end

  def down
    change_column :merge_request_diffs, :state, :string, null: false,
                  default: 'collected'
  end
end
Enable RuboCop for migrations Migrations shouldn't fail RuboCop checks - especially lint checks, such as the nested method check. To avoid changing code in existing migrations, add the magic comment to the top of each of them to skip that file. 2016-06-09 12:39:16 +00:00			`# rubocop:disable all`
Add version 4.2 to all existing migrations DB schema generated by a migration may look different in rails 4 and 5 (because rails 5 may use different default values). For this reason it's important to explicitly set for which rails version a migration was written for. See https://stackoverflow.com/questions/35929869/activerecordmigration-deprecation-warning-asks-for-rails-version-but-im-no/35930912#35930912 2018-11-13 07:27:31 +00:00			`class ChangeStateToAllowEmptyMergeRequestDiffs < ActiveRecord::Migration[4.2]`
Changed default in the database for the statemachine 2014-04-14 13:22:57 +00:00			`def up`
Fixed some hound notices 2014-04-14 13:24:37 +00:00			`change_column :merge_request_diffs, :state, :string, null: true,`
			`default: nil`
Changed default in the database for the statemachine 2014-04-14 13:22:57 +00:00			`end`

			`def down`
Fixed some hound notices 2014-04-14 13:24:37 +00:00			`change_column :merge_request_diffs, :state, :string, null: false,`
			`default: 'collected'`
Changed default in the database for the statemachine 2014-04-14 13:22:57 +00:00			`end`
			`end`