KEMBAR78
up_only in migration reports Rails/ReversibleMigration · Issue #5821 · rubocop/rubocop · GitHub
Skip to content

up_only in migration reports Rails/ReversibleMigration #5821

@morgoth

Description

@morgoth

Rails 5.2 introduced up_only method http://api.rubyonrails.org/classes/ActiveRecord/Migration.html#method-i-up_only

Rubocop reports Rails/ReversibleMigration violation for:

class DoSomething < ActiveRecord::Migration[5.1]
  def change
    up_only { execute("SET SESSION statement_timeout = 0;") }

    add_column :users, :name, :string
  end
end

Rails/ReversibleMigration: execute is not reversible

rubocop -v
0.55.0

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions