simplycop 2.34.6 → 2.34.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c1ab29bc46c57e4326c35f9e8ca117336f6f0fa92624676fba2cdec89f55170d
-  data.tar.gz: 87f6ddb8947813f55438735006641ed49773bd87a6802ce7e6061f74102047c8
+  metadata.gz: dbe92fb20c458fa89e0a5106707cb2f9b6b3850d2a3b4d80b28433979a412d6b
+  data.tar.gz: f98bd6ca357718dd5133ff77f3346c59c2b56a12e53cf52f5b27c9f0284ce658
 SHA512:
-  metadata.gz: 0dc12c7bad78a15542234d734e7d95ec23f10b5b3b38497f9221dc9ba71b55eba2d039e2a2345598c0f9d5965a25f022a6ada949c84f43fab4b523c78c35b996
-  data.tar.gz: ab37bd4125ba3f7804682addf9a0cfa0fa494fc54a9940003c0b9e269fda08850c16ff1547f62b05ed7b89cd40ab72329f45dee8c2c42e369e9d58c00d91d7e4
+  metadata.gz: a96d6c74ff3c75e94155962a96b156885c0913511314676ab89187df75bc370fe73f41806f107b33016e1c0a62e9391ed511c6018f366340491327459445eb0f
+  data.tar.gz: 48ae49c573b9b8007cb72414b621ed01e00a0dd850a7f19b1add758c85dab0f5a0c50ae21e6093df4de95e3784db8193ee57111e640a08eff2707833e654ed44

data/lib/simplycop/custom_cops/version_comparison.rb

@@ -9,8 +9,6 @@ module CustomCops
   #
   # If this cop flags a false positive, disable it with an inline rubocop comment.
   class VersionComparison < RuboCop::Cop::Base
-    extend RuboCop::Cop::AutoCorrector
-
     DISABLE_HINT = 'Disable if not a semantic version string.'
     MSG_TO_F = "Avoid `.to_f` on version strings; use `Gem::Version.new()`. #{DISABLE_HINT}"
     MSG_TO_I = "Avoid `.to_i` on version strings; use `Gem::Version.new()`. #{DISABLE_HINT}"
@@ -40,43 +38,26 @@ module CustomCops
       send(matcher, node) do |receiver|
         return unless version_related?(receiver)
 
-        add_offense(node, message: message) { |c| autocorrect_numeric(c, node, receiver) }
+        add_offense(node, message: message)
       end
     end
 
     def check_ordering_comparison(node)
-      ordering_comparison?(node) do |receiver, str_val|
+      ordering_comparison?(node) do |receiver, _str_val|
         return unless version_related?(receiver)
 
-        add_offense(node, message: MSG_STRING) do |c|
-          c.replace(node, build_comparison(receiver.source, node.method_name, "'#{str_val}'"))
-        end
+        add_offense(node, message: MSG_STRING)
       end
     end
 
     def check_reversed_ordering_comparison(node)
-      reversed_ordering_comparison?(node) do |str_val, receiver|
+      reversed_ordering_comparison?(node) do |_str_val, receiver|
         return unless version_related?(receiver)
 
-        add_offense(node, message: MSG_STRING) do |c|
-          c.replace(node, build_comparison(receiver.source, REVERSED_OPERATORS[node.method_name], "'#{str_val}'"))
-        end
-      end
-    end
-
-    def autocorrect_numeric(corrector, node, receiver)
-      parent = node.parent
-      if parent&.send_type? && ORDERING_OPERATORS.include?(parent.method_name)
-        corrector.replace(parent, build_comparison(receiver.source, parent.method_name, parent.arguments.first.source))
-      else
-        corrector.replace(node, "Gem::Version.new(#{receiver.source})")
+        add_offense(node, message: MSG_STRING)
       end
     end
 
-    def build_comparison(lhs, operator, rhs)
-      "Gem::Version.new(#{lhs}) #{operator} Gem::Version.new(#{rhs})"
-    end
-
     def version_related?(node)
       return false unless node
 

data/lib/simplycop/version.rb

@@ -7,5 +7,5 @@
 #
 
 module Simplycop
-  VERSION = '2.34.6'
+  VERSION = '2.34.7'
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: simplycop
 version: !ruby/object:Gem::Version
-  version: 2.34.6
+  version: 2.34.7
 platform: ruby
 authors:
 - Simply Business
 bindir: bin
 cert_chain: []
-date: 2026-01-07 00:00:00.000000000 Z
+date: 2026-01-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: benchmark