Merge pull request #935 from koic/make_style_inverse_methods_aware_of_active_support_methods

Make `Style/InverseMethods` aware of some Active Support methods

Author: koic

.rubocop.yml

@@ -66,6 +66,11 @@ Style/FormatStringToken:
   Exclude:
     - spec/**/*
 
+Style/InverseMethods:
+  # This rubocop-rails repository doesn't use Active Support, so it can't replace `include?` with `exclude?`.
+  InverseMethods:
+    :include?: ~
+
 Layout/HashAlignment:
   EnforcedHashRocketStyle:
     - key

changelog/change_make_style_inverse_methods_aware_of_active_support_methods.md

@@ -0,0 +1 @@
+* [#935](https://github.com/rubocop/rubocop-rails/pull/935): Make `Style/InverseMethods` aware of Active Support's `present?`, `blank?`, `include?`, and `exclude?` methods. ([@koic][])

config/default.yml

@@ -1144,6 +1144,14 @@ Style/FormatStringToken:
   AllowedMethods:
     - redirect
 
+Style/InverseMethods:
+  # `InverseMethods` are methods that can be inverted by a not (`not` or `!`)
+  # The relationship of inverse methods only needs to be defined in one direction.
+  # Keys and values both need to be defined as symbols.
+  InverseMethods:
+    :present?: :blank?
+    :include?: :exclude?
+
 Style/SymbolProc:
   AllowedMethods:
     - define_method

lib/rubocop/rails.rb

@@ -5,7 +5,7 @@ module RuboCop
   module Rails
     PROJECT_ROOT   = Pathname.new(__dir__).parent.parent.expand_path.freeze
     CONFIG_DEFAULT = PROJECT_ROOT.join('config', 'default.yml').freeze
-    CONFIG         = YAML.safe_load(CONFIG_DEFAULT.read).freeze
+    CONFIG         = YAML.safe_load(CONFIG_DEFAULT.read, permitted_classes: [Regexp, Symbol]).freeze
 
     private_constant(:CONFIG_DEFAULT, :PROJECT_ROOT)