rubocop-rails 2.4.0 → 2.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eac129cae1115b4038b4731039495db93945c73a041fb762d7412521d4ec3619
-  data.tar.gz: 5a68473da7c7799e223d0c5de4c10609751a633b6db573cb155d41c942dfb755
+  metadata.gz: c9d8cbc9fa9fc0752a334a92dd7af50f2a33331ac1b9604dc7c1f27efbc141e6
+  data.tar.gz: 81833c9b0dbc71254f75f18eb59a13147025efdff95342c3ba1855e773301fc2
 SHA512:
-  metadata.gz: 6f0efc72cd3569bfe93874a5f77113186e73947bc9d6665fbf926a4e9724e9e4390a312541b628d305c8ea733a94ce94212f38f80db71ec4be3d291d63029f1f
-  data.tar.gz: ae2d76dd0b58c09108b15cf83f363262904c698a8f0839532838f39d969b101885329228674ed237fcafc10a63732be8a6a8c2a1a800339bb7e4a4bd18fb759b
+  metadata.gz: a5ffc7a5e763076a13e5addebf2b4321ebfddbbb19e752bfe9697003eb4451e5d5eebc4ced8c2f3f55e1e10f24a81670e14b7ec48ee748d1c15996085618b5c8
+  data.tar.gz: 0e8b3557bfa819b2fb1304f992a48845afbb52db0c8c7beb970bbcc09c4345c5f9d2f9f273efb2692374f3a03c28c6d615824af20ca669c40359299dca53f41a

data/config/default.yml

@@ -2,7 +2,8 @@
 
 AllCops:
   Exclude:
-    - bin/bundle
+    - bin/*
+    - db/schema.rb
   # What version of Rails is the inspected code using?  If a value is specified
   # for TargetRailsVersion then it is used.  Acceptable values are specificed
   # as a float (i.e. 5.1); the patch version of Rails should not be included.
@@ -334,7 +335,7 @@ Rails/Present:
   UnlessBlank: true
 
 Rails/RakeEnvironment:
-  Description: 'Set `:environment` task as a dependency to all rake task.'
+  Description: 'Include `:environment` as a dependency for all Rake tasks.'
   Enabled: true
   Safe: false
   VersionAdded: '2.4'

data/lib/rubocop/cop/rails/belongs_to.rb

@@ -65,7 +65,7 @@ module RuboCop
           'In most configurations, this is the default and you can omit ' \
           'this option altogether'
 
-        def_node_matcher :match_belongs_to_with_options, <<-PATTERN
+        def_node_matcher :match_belongs_to_with_options, <<~PATTERN
           (send _ :belongs_to _
             (hash <$(pair (sym :required) ${true false}) ...>)
           )

data/lib/rubocop/cop/rails/blank.rb

@@ -63,7 +63,7 @@ module RuboCop
         # that have a single method call on the left hand side
         # (`bar || foo.empty?`) will blow up when checking
         # `(send (:nil) :== $_)`.
-        def_node_matcher :nil_or_empty?, <<-PATTERN
+        def_node_matcher :nil_or_empty?, <<~PATTERN
           (or
               {
                 (send $_ :!)
@@ -82,7 +82,7 @@ module RuboCop
 
         def_node_matcher :defining_blank?, '(def :blank? (args) ...)'
 
-        def_node_matcher :unless_present?, <<-PATTERN
+        def_node_matcher :unless_present?, <<~PATTERN
           (:if $(send $_ :present?) {nil? (...)} ...)
         PATTERN
 

data/lib/rubocop/cop/rails/bulk_change_table.rb

@@ -138,9 +138,9 @@ module RuboCop
 
           recorder = AlterMethodsRecorder.new
 
-          node.body.each_child_node(:send) do |send_node|
-            if combinable_alter_methods.include?(send_node.method_name)
-              recorder.process(send_node)
+          node.body.child_nodes.each do |child_node|
+            if call_to_combinable_alter_method? child_node
+              recorder.process(child_node)
             else
               recorder.flush
             end
@@ -218,6 +218,11 @@ module RuboCop
           end
         end
 
+        def call_to_combinable_alter_method?(child_node)
+          child_node.send_type? &&
+            combinable_alter_methods.include?(child_node.method_name)
+        end
+
         def combinable_alter_methods
           case database
           when MYSQL

data/lib/rubocop/cop/rails/create_table_with_timestamps.rb

@@ -43,22 +43,22 @@ module RuboCop
       class CreateTableWithTimestamps < Cop
         MSG = 'Add timestamps when creating a new table.'
 
-        def_node_matcher :create_table_with_block?, <<-PATTERN
+        def_node_matcher :create_table_with_block?, <<~PATTERN
           (block
             (send nil? :create_table ...)
             (args (arg _var))
             _)
         PATTERN
 
-        def_node_matcher :create_table_with_timestamps_proc?, <<-PATTERN
+        def_node_matcher :create_table_with_timestamps_proc?, <<~PATTERN
           (send nil? :create_table (sym _) ... (block-pass (sym :timestamps)))
         PATTERN
 
-        def_node_search :timestamps_included?, <<-PATTERN
+        def_node_search :timestamps_included?, <<~PATTERN
           (send _var :timestamps ...)
         PATTERN
 
-        def_node_search :created_at_or_updated_at_included?, <<-PATTERN
+        def_node_search :created_at_or_updated_at_included?, <<~PATTERN
           (send _var :datetime
             {(sym {:created_at :updated_at})(str {"created_at" "updated_at"})}
             ...)

data/lib/rubocop/cop/rails/date.rb

@@ -84,13 +84,13 @@ module RuboCop
         private
 
         def check_deprecated_methods(node)
-          DEPRECATED_METHODS.each do |relevant:, deprecated:|
-            next unless node.method?(deprecated.to_sym)
+          DEPRECATED_METHODS.each do |method|
+            next unless node.method?(method[:deprecated].to_sym)
 
             add_offense(node, location: :selector,
                               message: format(DEPRECATED_MSG,
-                                              deprecated: deprecated,
-                                              relevant: relevant))
+                                              deprecated: method[:deprecated],
+                                              relevant: method[:relevant]))
           end
         end
 

data/lib/rubocop/cop/rails/delegate.rb

@@ -55,7 +55,7 @@ module RuboCop
       class Delegate < Cop
         MSG = 'Use `delegate` to define delegations.'
 
-        def_node_matcher :delegate?, <<-PATTERN
+        def_node_matcher :delegate?, <<~PATTERN
           (def _method_name _args
             (send (send nil? _) _ ...))
         PATTERN

data/lib/rubocop/cop/rails/delegate_allow_blank.rb

@@ -16,7 +16,7 @@ module RuboCop
       class DelegateAllowBlank < Cop
         MSG = '`allow_blank` is not a valid option, use `allow_nil`.'
 
-        def_node_matcher :allow_blank_option, <<-PATTERN
+        def_node_matcher :allow_blank_option, <<~PATTERN
           (send nil? :delegate _ (hash <$(pair (sym :allow_blank) true) ...>))
         PATTERN
 

data/lib/rubocop/cop/rails/enum_hash.rb

@@ -42,7 +42,7 @@ module RuboCop
 
         def autocorrect(node)
           hash = node.children.each_with_index.map do |elem, index|
-            "#{elem.source} => #{index}"
+            "#{source(elem)} => #{index}"
           end.join(', ')
 
           ->(corrector) { corrector.replace(node.loc.expression, "{#{hash}}") }
@@ -58,6 +58,17 @@ module RuboCop
             key.source
           end
         end
+
+        def source(elem)
+          case elem.type
+          when :str
+            elem.value.dump
+          when :sym
+            elem.value.inspect
+          else
+            elem.source
+          end
+        end
       end
     end
   end

data/lib/rubocop/cop/rails/environment_comparison.rb

@@ -21,7 +21,7 @@ module RuboCop
         SYM_MSG = 'Do not compare `Rails.env` with a symbol, it will always ' \
           'evaluate to `false`.'
 
-        def_node_matcher :environment_str_comparison?, <<-PATTERN
+        def_node_matcher :environment_str_comparison?, <<~PATTERN
           (send
             (send (const {nil? cbase} :Rails) :env)
             :==
@@ -29,7 +29,7 @@ module RuboCop
           )
         PATTERN
 
-        def_node_matcher :environment_sym_comparison?, <<-PATTERN
+        def_node_matcher :environment_sym_comparison?, <<~PATTERN
           (send
             (send (const {nil? cbase} :Rails) :env)
             :==
@@ -59,7 +59,7 @@ module RuboCop
           "#{node.receiver.source}.#{content(node.first_argument)}?"
         end
 
-        def_node_matcher :content, <<-PATTERN
+        def_node_matcher :content, <<~PATTERN
           ({str sym} $_)
         PATTERN
       end

data/lib/rubocop/cop/rails/file_path.rb

@@ -34,15 +34,15 @@ module RuboCop
         MSG_ARGUMENTS = 'Please use `Rails.root.join(\'path\', \'to\')` ' \
                         'instead.'
 
-        def_node_matcher :file_join_nodes?, <<-PATTERN
+        def_node_matcher :file_join_nodes?, <<~PATTERN
           (send (const nil? :File) :join ...)
         PATTERN
 
-        def_node_search :rails_root_nodes?, <<-PATTERN
+        def_node_search :rails_root_nodes?, <<~PATTERN
           (send (const nil? :Rails) :root)
         PATTERN
 
-        def_node_matcher :rails_root_join_nodes?, <<-PATTERN
+        def_node_matcher :rails_root_join_nodes?, <<~PATTERN
           (send (send (const nil? :Rails) :root) :join ...)
         PATTERN
 

data/lib/rubocop/cop/rails/find_by.rb

@@ -19,7 +19,7 @@ module RuboCop
         MSG = 'Use `find_by` instead of `where.%<method>s`.'
         TARGET_SELECTORS = %i[first take].freeze
 
-        def_node_matcher :where_first?, <<-PATTERN
+        def_node_matcher :where_first?, <<~PATTERN
           (send ({send csend} _ :where ...) {:first :take})
         PATTERN
 

data/lib/rubocop/cop/rails/has_many_or_has_one_dependent.rb

@@ -23,27 +23,27 @@ module RuboCop
       class HasManyOrHasOneDependent < Cop
         MSG = 'Specify a `:dependent` option.'
 
-        def_node_search :active_resource_class?, <<-PATTERN
+        def_node_search :active_resource_class?, <<~PATTERN
           (const (const nil? :ActiveResource) :Base)
         PATTERN
 
-        def_node_matcher :association_without_options?, <<-PATTERN
+        def_node_matcher :association_without_options?, <<~PATTERN
           (send nil? {:has_many :has_one} _)
         PATTERN
 
-        def_node_matcher :association_with_options?, <<-PATTERN
+        def_node_matcher :association_with_options?, <<~PATTERN
           (send nil? {:has_many :has_one} _ (hash $...))
         PATTERN
 
-        def_node_matcher :dependent_option?, <<-PATTERN
+        def_node_matcher :dependent_option?, <<~PATTERN
           (pair (sym :dependent) !nil)
         PATTERN
 
-        def_node_matcher :present_option?, <<-PATTERN
+        def_node_matcher :present_option?, <<~PATTERN
           (pair (sym :through) !nil)
         PATTERN
 
-        def_node_matcher :with_options_block, <<-PATTERN
+        def_node_matcher :with_options_block, <<~PATTERN
           (block
             (send nil? :with_options
               (hash $...))

data/lib/rubocop/cop/rails/http_positional_arguments.rb

@@ -28,7 +28,7 @@ module RuboCop
 
         minimum_target_rails_version 5.0
 
-        def_node_matcher :http_request?, <<-PATTERN
+        def_node_matcher :http_request?, <<~PATTERN
           (send nil? {#{HTTP_METHODS.map(&:inspect).join(' ')}} !nil? $_ ...)
         PATTERN
 

data/lib/rubocop/cop/rails/http_status.rb

@@ -34,14 +34,14 @@ module RuboCop
       class HttpStatus < Cop
         include ConfigurableEnforcedStyle
 
-        def_node_matcher :http_status, <<-PATTERN
+        def_node_matcher :http_status, <<~PATTERN
           {
             (send nil? {:render :redirect_to} _ $hash)
             (send nil? {:render :redirect_to} $hash)
           }
         PATTERN
 
-        def_node_matcher :status_code, <<-PATTERN
+        def_node_matcher :status_code, <<~PATTERN
           (hash <(pair (sym :status) ${int sym}) ...>)
         PATTERN
 

data/lib/rubocop/cop/rails/ignored_skip_action_filter_option.rb

@@ -49,7 +49,7 @@ module RuboCop
           :skip_action_callback
         ].freeze
 
-        def_node_matcher :filter_options, <<-PATTERN
+        def_node_matcher :filter_options, <<~PATTERN
           (send
             nil?
             {#{FILTERS.join(' ')}}

data/lib/rubocop/cop/rails/inverse_of.rb

@@ -137,39 +137,39 @@ module RuboCop
         NIL_MSG = 'You specified `inverse_of: nil`, you probably meant to ' \
           'use `inverse_of: false`.'
 
-        def_node_matcher :association_recv_arguments, <<-PATTERN
+        def_node_matcher :association_recv_arguments, <<~PATTERN
           (send $_ {:has_many :has_one :belongs_to} _ $...)
         PATTERN
 
-        def_node_matcher :options_from_argument, <<-PATTERN
+        def_node_matcher :options_from_argument, <<~PATTERN
           (hash $...)
         PATTERN
 
-        def_node_matcher :conditions_option?, <<-PATTERN
+        def_node_matcher :conditions_option?, <<~PATTERN
           (pair (sym :conditions) !nil)
         PATTERN
 
-        def_node_matcher :through_option?, <<-PATTERN
+        def_node_matcher :through_option?, <<~PATTERN
           (pair (sym :through) !nil)
         PATTERN
 
-        def_node_matcher :polymorphic_option?, <<-PATTERN
+        def_node_matcher :polymorphic_option?, <<~PATTERN
           (pair (sym :polymorphic) !nil)
         PATTERN
 
-        def_node_matcher :as_option?, <<-PATTERN
+        def_node_matcher :as_option?, <<~PATTERN
           (pair (sym :as) !nil)
         PATTERN
 
-        def_node_matcher :foreign_key_option?, <<-PATTERN
+        def_node_matcher :foreign_key_option?, <<~PATTERN
           (pair (sym :foreign_key) !nil)
         PATTERN
 
-        def_node_matcher :inverse_of_option?, <<-PATTERN
+        def_node_matcher :inverse_of_option?, <<~PATTERN
           (pair (sym :inverse_of) !nil)
         PATTERN
 
-        def_node_matcher :inverse_of_nil_option?, <<-PATTERN
+        def_node_matcher :inverse_of_nil_option?, <<~PATTERN
           (pair (sym :inverse_of) nil)
         PATTERN
 

data/lib/rubocop/cop/rails/lexically_scoped_action_filter.rb

@@ -101,7 +101,7 @@ module RuboCop
           :skip_action_callback
         ].freeze
 
-        def_node_matcher :only_or_except_filter_methods, <<-PATTERN
+        def_node_matcher :only_or_except_filter_methods, <<~PATTERN
           (send
             nil?
             {#{FILTERS.join(' ')}}

data/lib/rubocop/cop/rails/link_to_blank.rb

@@ -23,15 +23,15 @@ module RuboCop
       class LinkToBlank < Cop
         MSG = 'Specify a `:rel` option containing noopener.'
 
-        def_node_matcher :blank_target?, <<-PATTERN
+        def_node_matcher :blank_target?, <<~PATTERN
           (pair {(sym :target) (str "target")} {(str "_blank") (sym :_blank)})
         PATTERN
 
-        def_node_matcher :includes_noopener?, <<-PATTERN
+        def_node_matcher :includes_noopener?, <<~PATTERN
           (pair {(sym :rel) (str "rel")} ({str sym} #contains_noopener?))
         PATTERN
 
-        def_node_matcher :rel_node?, <<-PATTERN
+        def_node_matcher :rel_node?, <<~PATTERN
           (pair {(sym :rel) (str "rel")} (str _))
         PATTERN
 

data/lib/rubocop/cop/rails/not_null_column.rb

@@ -19,19 +19,19 @@ module RuboCop
       class NotNullColumn < Cop
         MSG = 'Do not add a NOT NULL column without a default value.'
 
-        def_node_matcher :add_not_null_column?, <<-PATTERN
+        def_node_matcher :add_not_null_column?, <<~PATTERN
           (send nil? :add_column _ _ _ (hash $...))
         PATTERN
 
-        def_node_matcher :add_not_null_reference?, <<-PATTERN
+        def_node_matcher :add_not_null_reference?, <<~PATTERN
           (send nil? :add_reference _ _ (hash $...))
         PATTERN
 
-        def_node_matcher :null_false?, <<-PATTERN
+        def_node_matcher :null_false?, <<~PATTERN
           (pair (sym :null) (false))
         PATTERN
 
-        def_node_matcher :default_option?, <<-PATTERN
+        def_node_matcher :default_option?, <<~PATTERN
           (pair (sym :default) !nil)
         PATTERN
 

data/lib/rubocop/cop/rails/output.rb

@@ -17,11 +17,11 @@ module RuboCop
         MSG = 'Do not write to stdout. ' \
               "Use Rails's logger if you want to log."
 
-        def_node_matcher :output?, <<-PATTERN
+        def_node_matcher :output?, <<~PATTERN
           (send nil? {:ap :p :pp :pretty_print :print :puts} ...)
         PATTERN
 
-        def_node_matcher :io_output?, <<-PATTERN
+        def_node_matcher :io_output?, <<~PATTERN
           (send
             {
               (gvar #match_gvar?)

data/lib/rubocop/cop/rails/presence.rb

@@ -42,7 +42,7 @@ module RuboCop
 
         MSG = 'Use `%<prefer>s` instead of `%<current>s`.'
 
-        def_node_matcher :redundant_receiver_and_other, <<-PATTERN
+        def_node_matcher :redundant_receiver_and_other, <<~PATTERN
           {
             (if
               (send $_recv :present?)
@@ -57,7 +57,7 @@ module RuboCop
           }
         PATTERN
 
-        def_node_matcher :redundant_negative_receiver_and_other, <<-PATTERN
+        def_node_matcher :redundant_negative_receiver_and_other, <<~PATTERN
           {
             (if
               (send (send $_recv :present?) :!)

data/lib/rubocop/cop/rails/present.rb

@@ -50,7 +50,7 @@ module RuboCop
         MSG_UNLESS_BLANK = 'Use `if %<prefer>s` instead of ' \
                            '`%<current>s`.'
 
-        def_node_matcher :exists_and_not_empty?, <<-PATTERN
+        def_node_matcher :exists_and_not_empty?, <<~PATTERN
           (and
               {
                 (send (send $_ :nil?) :!)
@@ -66,7 +66,7 @@ module RuboCop
 
         def_node_matcher :not_blank?, '(send (send $_ :blank?) :!)'
 
-        def_node_matcher :unless_blank?, <<-PATTERN
+        def_node_matcher :unless_blank?, <<~PATTERN
           (:if $(send $_ :blank?) {nil? (...)} ...)
         PATTERN
 

data/lib/rubocop/cop/rails/rake_environment.rb

@@ -3,16 +3,16 @@
 module RuboCop
   module Cop
     module Rails
-      # This cop checks rake task definition without `environment` dependency.
-      # `environment` dependency is important because it loads application code
-      # for the rake task. The rake task cannot use application code, such as
-      # models, without `environment` dependency.
+      # This cop checks for Rake tasks without the `:environment` task
+      # dependency. The `:environment` task loads application code for other
+      # Rake tasks. Without it, tasks cannot make use of application code like
+      # models.
       #
       # You can ignore the offense if the task satisfies at least one of the
       # following conditions:
       #
       # * The task does not need application code.
-      # * The task invokes :environment task.
+      # * The task invokes the `:environment` task.
       #
       # @example
       #   # bad
@@ -26,9 +26,9 @@ module RuboCop
       #   end
       #
       class RakeEnvironment < Cop
-        MSG = 'Set `:environment` task as a dependency to all rake task.'
+        MSG = 'Include `:environment` task as a dependency for all Rake tasks.'
 
-        def_node_matcher :task_definition?, <<-PATTERN
+        def_node_matcher :task_definition?, <<~PATTERN
           (send nil? :task ...)
         PATTERN
 

data/lib/rubocop/cop/rails/read_write_attribute.rb

@@ -26,7 +26,7 @@ module RuboCop
       class ReadWriteAttribute < Cop
         MSG = 'Prefer `%<prefer>s` over `%<current>s`.'
 
-        def_node_matcher :read_write_attribute?, <<-PATTERN
+        def_node_matcher :read_write_attribute?, <<~PATTERN
           {
             (send nil? :read_attribute _)
             (send nil? :write_attribute _ _)

data/lib/rubocop/cop/rails/redundant_receiver_in_with_options.rb

@@ -62,7 +62,7 @@ module RuboCop
 
         MSG = 'Redundant receiver in `with_options`.'
 
-        def_node_matcher :with_options?, <<-PATTERN
+        def_node_matcher :with_options?, <<~PATTERN
           (block
             (send nil? :with_options
               (...))
@@ -71,11 +71,11 @@ module RuboCop
             $_body)
         PATTERN
 
-        def_node_search :all_block_nodes_in, <<-PATTERN
+        def_node_search :all_block_nodes_in, <<~PATTERN
           (block ...)
         PATTERN
 
-        def_node_search :all_send_nodes_in, <<-PATTERN
+        def_node_search :all_send_nodes_in, <<~PATTERN
           (send ...)
         PATTERN
 

data/lib/rubocop/cop/rails/reflection_class_name.rb

@@ -16,13 +16,13 @@ module RuboCop
       class ReflectionClassName < Cop
         MSG = 'Use a string value for `class_name`.'
 
-        def_node_matcher :association_with_reflection, <<-PATTERN
+        def_node_matcher :association_with_reflection, <<~PATTERN
           (send nil? {:has_many :has_one :belongs_to} _
             (hash <$#reflection_class_name ...>)
           )
         PATTERN
 
-        def_node_matcher :reflection_class_name, <<-PATTERN
+        def_node_matcher :reflection_class_name, <<~PATTERN
           (pair (sym :class_name) [!dstr !str !sym])
         PATTERN
 

data/lib/rubocop/cop/rails/relative_date_constant.rb

@@ -14,6 +14,15 @@ module RuboCop
       #
       #   # good
       #   class SomeClass
+      #     EXPIRES = 1.week
+      #
+      #     def self.expired_at
+      #       EXPIRES.since
+      #     end
+      #   end
+      #
+      #   # good
+      #   class SomeClass
       #     def self.expired_at
       #       1.week.since
       #     end
@@ -68,7 +77,7 @@ module RuboCop
 
         private
 
-        def_node_matcher :relative_date_assignment?, <<-PATTERN
+        def_node_matcher :relative_date_assignment?, <<~PATTERN
           {
             (casgn _ _ (send _ ${:since :from_now :after :ago :until :before}))
             (casgn _ _ ({erange irange} _ (send _ ${:since :from_now :after :ago :until :before})))
@@ -76,11 +85,11 @@ module RuboCop
           }
         PATTERN
 
-        def_node_matcher :relative_date_or_assignment?, <<-PATTERN
+        def_node_matcher :relative_date_or_assignment?, <<~PATTERN
           (:or_asgn (casgn _ _) (send _ ${:since :from_now :after :ago :until :before}))
         PATTERN
 
-        def_node_matcher :relative_date?, <<-PATTERN
+        def_node_matcher :relative_date?, <<~PATTERN
           {
             ({erange irange} _ (send _ ${:since :from_now :after :ago :until :before}))
             ({erange irange} (send _ ${:since :from_now :after :ago :until :before}) _)

data/lib/rubocop/cop/rails/request_referer.rb

@@ -25,7 +25,7 @@ module RuboCop
         MSG = 'Use `request.%<prefer>s` instead of ' \
               '`request.%<current>s`.'
 
-        def_node_matcher :referer?, <<-PATTERN
+        def_node_matcher :referer?, <<~PATTERN
           (send (send nil? :request) {:referer :referrer})
         PATTERN
 

data/lib/rubocop/cop/rails/reversible_migration.rb

@@ -128,23 +128,23 @@ module RuboCop
       class ReversibleMigration < Cop
         MSG = '%<action>s is not reversible.'
 
-        def_node_matcher :irreversible_schema_statement_call, <<-PATTERN
+        def_node_matcher :irreversible_schema_statement_call, <<~PATTERN
           (send nil? ${:execute :remove_belongs_to} ...)
         PATTERN
 
-        def_node_matcher :drop_table_call, <<-PATTERN
+        def_node_matcher :drop_table_call, <<~PATTERN
           (send nil? :drop_table ...)
         PATTERN
 
-        def_node_matcher :remove_column_call, <<-PATTERN
+        def_node_matcher :remove_column_call, <<~PATTERN
           (send nil? :remove_column $...)
         PATTERN
 
-        def_node_matcher :remove_foreign_key_call, <<-PATTERN
+        def_node_matcher :remove_foreign_key_call, <<~PATTERN
           (send nil? :remove_foreign_key _ $_)
         PATTERN
 
-        def_node_matcher :change_table_call, <<-PATTERN
+        def_node_matcher :change_table_call, <<~PATTERN
           (send nil? :change_table $_ ...)
         PATTERN
 

data/lib/rubocop/cop/rails/safe_navigation.rb

@@ -41,7 +41,7 @@ module RuboCop
 
         MSG = 'Use safe navigation (`&.`) instead of `%<try>s`.'
 
-        def_node_matcher :try_call, <<-PATTERN
+        def_node_matcher :try_call, <<~PATTERN
           (send !nil? ${:try :try!} $_ ...)
         PATTERN
 

data/lib/rubocop/cop/rails/safe_navigation_with_blank.rb

@@ -24,7 +24,7 @@ module RuboCop
           'Avoid calling `blank?` with the safe navigation operator ' \
           'in conditionals.'
 
-        def_node_matcher :safe_navigation_blank_in_conditional?, <<-PATTERN
+        def_node_matcher :safe_navigation_blank_in_conditional?, <<~PATTERN
           (if $(csend ... :blank?) ...)
         PATTERN
 

data/lib/rubocop/cop/rails/skips_model_validations.rb

@@ -49,7 +49,7 @@ module RuboCop
                                     update_columns
                                     update_counters].freeze
 
-        def_node_matcher :good_touch?, <<-PATTERN
+        def_node_matcher :good_touch?, <<~PATTERN
           (send (const nil? :FileUtils) :touch ...)
         PATTERN
 

data/lib/rubocop/cop/rails/unknown_env.rb

@@ -31,7 +31,7 @@ module RuboCop
           (send #rails_env? $#unknown_env_predicate?)
         PATTERN
 
-        def_node_matcher :unknown_environment_equal?, <<-PATTERN
+        def_node_matcher :unknown_environment_equal?, <<~PATTERN
           {
             (send #rails_env? {:== :===} $(str #unknown_env_name?))
             (send $(str #unknown_env_name?) {:== :===} #rails_env?)

data/lib/rubocop/rails/version.rb

@@ -4,7 +4,7 @@ module RuboCop
   module Rails
     # This module holds the RuboCop Rails version information.
     module Version
-      STRING = '2.4.0'
+      STRING = '2.4.1'
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubocop-rails
 version: !ruby/object:Gem::Version
-  version: 2.4.0
+  version: 2.4.1
 platform: ruby
 authors:
 - Bozhidar Batsov
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-11-27 00:00:00.000000000 Z
+date: 2019-12-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -40,9 +40,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.72.0
-description: |2
-      Automatic Rails code style checking tool.
-      A RuboCop extension focused on enforcing Rails best practices and coding conventions.
+description: |
+  Automatic Rails code style checking tool.
+  A RuboCop extension focused on enforcing Rails best practices and coding conventions.
 email: rubocop@googlegroups.com
 executables: []
 extensions: []
@@ -141,7 +141,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Automatic Rails code style checking tool.