Make Lint/RedundantSafeNavigation aware of several methods

This PR makes `Lint/RedundantSafeNavigation` aware of `blank?`, `presence`, and `present?` methods.
rubocop · May 27, 2023 · 4052e55 · 4052e55
1 parent b70b1d8
commit 4052e55
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/changelog/change_make_lint_redundant_safe_navigation_aware_of_several_methods.md b/changelog/change_make_lint_redundant_safe_navigation_aware_of_several_methods.md
@@ -0,0 +1 @@
+* [#1014](https://github.com/rubocop/rubocop-rails/pull/1014): Make `Lint/RedundantSafeNavigation` aware of `blank?`, `presence`, and `present?` methods. ([@koic][])
diff --git a/config/default.yml b/config/default.yml
@@ -48,6 +48,20 @@ Lint/NumberConversion:
     - in_milliseconds
   AllowedPatterns: []
 
+Lint/RedundantSafeNavigation:
+  # Add `blank?`, `presence`, and `present?` methods to the default of the RuboCop core.
+  # https://github.com/rubocop/rubocop/blob/v1.51.0/config/default.yml#L2148-L2159
+  AllowedMethods:
+    - instance_of?
+    - kind_of?
+    - is_a?
+    - eql?
+    - respond_to?
+    - equal?
+    - blank?
+    - presence
+    - present?
+
 Rails:
   Enabled: true
   DocumentationBaseURL: https://docs.rubocop.org/rubocop-rails