Merge pull request #18577 from Homebrew/dependabot/bundler/Library/Ho…

…mebrew/rubocop-1.67.0
Homebrew · Oct 18, 2024 · fdd2aa7 · fdd2aa7
2 parents 215064e + 7aee21c
commit fdd2aa7
Show file tree

Hide file tree

Showing 6 changed files with 1,061 additions and 991 deletions.
diff --git a/Library/Homebrew/Gemfile.lock b/Library/Homebrew/Gemfile.lock
@@ -72,7 +72,7 @@ GEM
     rspec-support (3.13.1)
     rspec_junit_formatter (0.6.0)
       rspec-core (>= 2, < 4, != 2.12.0)
-    rubocop (1.66.1)
+    rubocop (1.67.0)
       json (~> 2.3)
       language_server-protocol (>= 3.17.0)
       parallel (~> 1.10)

diff --git a/Library/Homebrew/extend/hash/keys.rb b/Library/Homebrew/extend/hash/keys.rb
@@ -119,7 +119,7 @@ def _deep_transform_keys_in_object!(object, &block)
     case object
     when Hash
       # We can't use `each_key` here because we're updating the hash in-place.
-      object.keys.each do |key| # rubocop:disable Style/HashEachMethods
+      object.keys.each do |key|
         value = object.delete(key)
         object[yield(key)] = _deep_transform_keys_in_object!(value, &block)
       end

diff --git a/Library/Homebrew/rubocops/lines.rb b/Library/Homebrew/rubocops/lines.rb
@@ -338,8 +338,7 @@ def audit_formula(formula_nodes)
               good_args = "Utils.#{command}({ \"#{match[1]}\" => \"#{match[2]}\" }, \"#{match[3]}\")"
 
               problem "Use `#{good_args}` instead of `#{method.source}`" do |corrector|
-                corrector.replace(@offensive_node.source_range,
-                                  "{ \"#{match[1]}\" => \"#{match[2]}\" }, \"#{match[3]}\"")
+                corrector.replace(method.source_range, good_args)
               end
             end
           end