Fix AllowRegexpMatch of `Performance/RedundantEqualityComparisonBlo…

…ck` by default This PR fixes `AllowRegexpMatch` of `Performance/RedundantEqualityComparisonBlock` by default. Default value is true as documented example below: https://docs.rubocop.org/rubocop-performance/cops_performance.html#performanceredundantequalitycomparisonblock
rubocop · Apr 9, 2023 · 105060c · 105060c
1 parent c6abfdd
commit 105060c
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/changelog/fix_default_config_of_redundant_equality_comparison_block.md b/changelog/fix_default_config_of_redundant_equality_comparison_block.md
@@ -0,0 +1 @@
+* [#913](https://github.com/rubocop/rubocop-performance/pull/931): Fix the default config for `AllowRegexpMatch` option of `Performance/RedundantEqualityComparisonBlock`. ([@koic][])
diff --git a/config/default.yml b/config/default.yml
@@ -226,7 +226,7 @@ Performance/RedundantEqualityComparisonBlock:
   Reference: 'https://github.com/rails/rails/pull/41363'
   Enabled: pending
   Safe: false
-  AllowRegexpMatch: false
+  AllowRegexpMatch: true
   VersionAdded: '1.10'
 
 Performance/RedundantMatch:

diff --git a/lib/rubocop/cop/performance/redundant_equality_comparison_block.rb b/lib/rubocop/cop/performance/redundant_equality_comparison_block.rb
@@ -10,7 +10,7 @@ module Performance
       # behavior is appropriately overridden in subclass. For example,
       # `Range#===` returns `true` when argument is within the range.
       #
-      # This cop has `AllowRegexpMatch` option and it is false by default because
+      # This cop has `AllowRegexpMatch` option and it is true by default because
       # `regexp.match?('string')` often used in block changes to the opposite result:
       #
       # [source,ruby]