Merge pull request #541 from Shopify/dependabot/bundler/rubocop-1.51.0

Bump rubocop from 1.50.2 to 1.51.0

Author: sambostock

Gemfile.lock

@@ -2,7 +2,7 @@ PATH
   remote: .
   specs:
     rubocop-shopify (2.13.0)
-      rubocop (~> 1.50)
+      rubocop (~> 1.51)
 
 GEM
   remote: https://rubygems.org/
@@ -14,8 +14,8 @@ GEM
     json (2.6.3)
     method_source (1.0.0)
     minitest (5.18.0)
-    parallel (1.22.1)
-    parser (3.2.2.0)
+    parallel (1.23.0)
+    parser (3.2.2.1)
       ast (~> 2.4.1)
     pry (0.14.1)
       coderay (~> 1.1)
@@ -27,7 +27,7 @@ GEM
     rake (13.0.6)
     regexp_parser (2.8.0)
     rexml (3.2.5)
-    rubocop (1.50.2)
+    rubocop (1.51.0)
       json (~> 2.3)
       parallel (~> 1.10)
       parser (>= 3.2.0.0)
@@ -37,7 +37,7 @@ GEM
       rubocop-ast (>= 1.28.0, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 2.4.0, < 3.0)
-    rubocop-ast (1.28.0)
+    rubocop-ast (1.28.1)
       parser (>= 3.2.1.0)
     ruby-progressbar (1.13.0)
     unicode-display_width (2.4.2)

rubocop-shopify.gemspec

@@ -23,5 +23,5 @@ Gem::Specification.new do |s|
 
   s.required_ruby_version = ">= 2.7.0"
 
-  s.add_dependency("rubocop", "~> 1.50")
+  s.add_dependency("rubocop", "~> 1.51")
 end

rubocop.yml

@@ -513,6 +513,9 @@ Style/EnvHome:
 Style/EvalWithLocation:
   Enabled: false
 
+Style/ExactRegexpMatch:
+  Enabled: true
+
 Style/ExpandPathArguments:
   Enabled: false
 

test/fixtures/full_config.yml

@@ -94,6 +94,9 @@ AllCops:
     - graphql
     rubocop-capybara:
     - capybara
+    rubocop-factory_bot:
+    - factory_bot
+    - factory_bot_rails
   ActiveSupportExtensionsEnabled: false
 Bundler/DuplicatedGem:
   Description: Checks for duplicate gem entries in Gemfile.
@@ -1726,6 +1729,8 @@ Lint/TopLevelReturnWithArgument:
   Description: Detects top level return statements with argument.
   Enabled: false
   VersionAdded: '0.89'
+  Exclude:
+  - "/**/*.jb"
 Lint/TrailingCommaInAttributeDeclaration:
   Description: Checks for trailing commas in attribute declarations.
   Enabled: false
@@ -1818,6 +1823,8 @@ Lint/UselessAssignment:
   StyleGuide: "#underscore-unused-vars"
   Enabled: true
   VersionAdded: '0.11'
+  VersionChanged: '1.51'
+  SafeAutoCorrect: false
 Lint/UselessElseWithoutRescue:
   Description: Checks for useless `else` in `begin..end` without `rescue`.
   Enabled: true
@@ -2226,7 +2233,9 @@ Security/CompoundHash:
   Description: When overwriting Object#hash to combine values, prefer delegating to
     Array#hash over writing a custom implementation.
   Enabled: false
+  Safe: false
   VersionAdded: '1.28'
+  VersionChanged: '1.51'
 Security/Eval:
   Description: The use of eval represents a serious security risk.
   Enabled: true
@@ -2595,7 +2604,9 @@ Style/DataInheritance:
   Description: Checks for inheritance from Data.define.
   StyleGuide: "#no-extend-data-define"
   Enabled: true
+  SafeAutoCorrect: false
   VersionAdded: '1.49'
+  VersionChanged: '1.51'
 Style/DateTime:
   Description: Use Time over DateTime.
   StyleGuide: "#date-time"
@@ -2754,6 +2765,10 @@ Style/EvenOdd:
   Enabled: true
   VersionAdded: '0.12'
   VersionChanged: '0.29'
+Style/ExactRegexpMatch:
+  Description: Checks for exact regexp match inside Regexp literals.
+  Enabled: true
+  VersionAdded: '1.51'
 Style/ExpandPathArguments:
   Description: Use `expand_path(__dir__)` instead of `expand_path('..', __FILE__)`.
   Enabled: false
@@ -3473,7 +3488,9 @@ Style/OpenStructUse:
   Reference:
   - https://docs.ruby-lang.org/en/3.0.0/OpenStruct.html#class-OpenStruct-label-Caveats
   Enabled: true
+  Safe: false
   VersionAdded: '1.23'
+  VersionChanged: '1.51'
 Style/OperatorMethodCall:
   Description: Checks for redundant dot before operator method call.
   StyleGuide: "#operator-method-call"