Update rubocop requirement from ~> 1.75.0 to ~> 1.76.1 (#314)

dependabot[bot] · ntkme · web-flow · commit 22221e1103c6 · 2025-06-09T09:58:12.000-07:00
* Update rubocop requirement from ~> 1.75.0 to ~> 1.76.1 Updates the requirements on [rubocop](https://github.com/rubocop/rubocop) to permit the latest version. - [Release notes](https://github.com/rubocop/rubocop/releases) - [Changelog](https://github.com/rubocop/rubocop/blob/master/CHANGELOG.md) - [Commits](rubocop/rubocop@v1.75.0...v1.76.1) --- updated-dependencies: - dependency-name: rubocop dependency-version: 1.76.1 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> * Fix lint --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: なつき <i@ntk.me>
diff --git a/Gemfile b/Gemfile
@@ -7,7 +7,7 @@ gemspec
 group :development do
   gem 'rake', '>= 13'
   gem 'rspec', '~> 3.13.0'
-  gem 'rubocop', '~> 1.75.0'
+  gem 'rubocop', '~> 1.76.1'
   gem 'rubocop-performance', '~> 1.25.0'
   gem 'rubocop-rake', '~> 0.7.1'
   gem 'rubocop-rspec', '~> 3.6.0'
diff --git a/lib/sass/value.rb b/lib/sass/value.rb
@@ -38,7 +38,7 @@ def to_a
     end
 
     # @return [::Boolean]
-    def to_bool
+    def to_bool # rubocop:disable Naming/PredicateMethod
       true
     end
 
diff --git a/lib/sass/value/color.rb b/lib/sass/value/color.rb
@@ -318,21 +318,21 @@ def ==(other)
 
         if legacy?
           return false unless other.legacy?
-          return false unless FuzzyMath.equals_nilable(other.alpha_or_nil, alpha_or_nil)
+          return false unless FuzzyMath.equals_nilable?(other.alpha_or_nil, alpha_or_nil)
 
           if _space == other._space
-            FuzzyMath.equals_nilable(other.channel0_or_nil, channel0_or_nil) &&
-              FuzzyMath.equals_nilable(other.channel1_or_nil, channel1_or_nil) &&
-              FuzzyMath.equals_nilable(other.channel2_or_nil, channel2_or_nil)
+            FuzzyMath.equals_nilable?(other.channel0_or_nil, channel0_or_nil) &&
+              FuzzyMath.equals_nilable?(other.channel1_or_nil, channel1_or_nil) &&
+              FuzzyMath.equals_nilable?(other.channel2_or_nil, channel2_or_nil)
           else
             _to_space(Space::RGB) == other._to_space(Space::RGB)
           end
         else
           other._space == _space &&
-            FuzzyMath.equals_nilable(other.channel0_or_nil, channel0_or_nil) &&
-            FuzzyMath.equals_nilable(other.channel1_or_nil, channel1_or_nil) &&
-            FuzzyMath.equals_nilable(other.channel2_or_nil, channel2_or_nil) &&
-            FuzzyMath.equals_nilable(other.alpha_or_nil, alpha_or_nil)
+            FuzzyMath.equals_nilable?(other.channel0_or_nil, channel0_or_nil) &&
+            FuzzyMath.equals_nilable?(other.channel1_or_nil, channel1_or_nil) &&
+            FuzzyMath.equals_nilable?(other.channel2_or_nil, channel2_or_nil) &&
+            FuzzyMath.equals_nilable?(other.alpha_or_nil, alpha_or_nil)
         end
       end
 
@@ -367,9 +367,9 @@ def channel0_missing?
       def channel0_powerless?
         case _space
         when Space::HSL
-          FuzzyMath.equals(channel1, 0)
+          FuzzyMath.equals?(channel1, 0)
         when Space::HWB
-          FuzzyMath.greater_than_or_equals(channel1 + channel2, 100)
+          FuzzyMath.greater_than_or_equals?(channel1 + channel2, 100)
         else
           false
         end
@@ -398,7 +398,7 @@ def channel2_missing?
       def channel2_powerless?
         case _space
         when Space::LCH, Space::OKLCH
-          FuzzyMath.equals(channel1, 0)
+          FuzzyMath.equals?(channel1, 0)
         else
           false
         end
@@ -449,7 +449,7 @@ def _initialize_for_space_internal(space, channel0, channel1, channel2, alpha =
         when Space::HSL
           _initialize_for_space(
             space,
-            _normalize_hue(channel0, invert: !channel1.nil? && FuzzyMath.less_than(channel1, 0)),
+            _normalize_hue(channel0, invert: !channel1.nil? && FuzzyMath.less_than?(channel1, 0)),
             channel1&.abs,
             channel2,
             alpha
@@ -461,7 +461,7 @@ def _initialize_for_space_internal(space, channel0, channel1, channel2, alpha =
             space,
             channel0,
             channel1&.abs,
-            _normalize_hue(channel2, invert: !channel1.nil? && FuzzyMath.less_than(channel1, 0)),
+            _normalize_hue(channel2, invert: !channel1.nil? && FuzzyMath.less_than?(channel1, 0)),
             alpha
           )
         else
@@ -488,7 +488,7 @@ def _normalize_hue(hue, invert:)
       def _is_channel_in_gamut(value, channel)
         case channel
         when LinearChannel
-          FuzzyMath.less_than_or_equals(value, channel.max) && FuzzyMath.greater_than_or_equals(value, channel.min)
+          FuzzyMath.less_than_or_equals?(value, channel.max) && FuzzyMath.greater_than_or_equals?(value, channel.min)
         else
           true
         end
@@ -500,8 +500,8 @@ def _interpolate(other, method, weight: nil)
           raise Sass::ScriptError.new("Expected #{wieght} to be within 0 and 1.", 'weight')
         end
 
-        return other if FuzzyMath.equals(weight, 0)
-        return self if FuzzyMath.equals(weight, 1)
+        return other if FuzzyMath.equals?(weight, 0)
+        return self if FuzzyMath.equals?(weight, 1)
 
         color1 = _to_space(method.space)
         color2 = other._to_space(method.space)
diff --git a/lib/sass/value/color/gamut_map_method/local_minde.rb b/lib/sass/value/color/gamut_map_method/local_minde.rb
@@ -28,7 +28,7 @@ def map(color)
             hue = original_oklch.send(:channel2_or_nil)
             alpha = original_oklch.send(:alpha_or_nil)
 
-            if FuzzyMath.greater_than_or_equals(lightness.nil? ? 0 : lightness, 1)
+            if FuzzyMath.greater_than_or_equals?(lightness.nil? ? 0 : lightness, 1)
               if color.legacy?
                 return Color.send(:_for_space,
                                   Space::RGB, 255, 255, 255, color.send(:alpha_or_nil))
@@ -37,7 +37,7 @@ def map(color)
                 return Color.send(:for_space_internal,
                                   color.send(:_space), 1, 1, 1, color.send(:alpha_or_nil))
               end
-            elsif FuzzyMath.less_than_or_equals(lightness.nil? ? 0 : lightness, 0)
+            elsif FuzzyMath.less_than_or_equals?(lightness.nil? ? 0 : lightness, 0)
               return Color.send(:_for_space,
                                 Space::RGB, 0, 0, 0, color.send(:alpha_or_nil))
                           .send(:_to_space, color.send(:_space))
diff --git a/lib/sass/value/color/space/lab.rb b/lib/sass/value/color/space/lab.rb
@@ -24,7 +24,7 @@ def convert(dest, lightness, a, b, alpha, # rubocop:disable Naming/MethodParamet
                       missing_chroma: false, missing_hue: false)
             case dest
             when LAB
-              powerless_ab = lightness.nil? || FuzzyMath.equals(lightness, 0)
+              powerless_ab = lightness.nil? || FuzzyMath.equals?(lightness, 0)
               Color.send(
                 :_for_space,
                 dest,
diff --git a/lib/sass/value/color/space/srgb.rb b/lib/sass/value/color/space/srgb.rb
@@ -55,7 +55,7 @@ def convert(dest, red, green, blue, alpha,
                 Color.send(
                   :for_space_internal,
                   dest,
-                  missing_hue || FuzzyMath.equals(saturation, 0) ? nil : hue % 360,
+                  missing_hue || FuzzyMath.equals?(saturation, 0) ? nil : hue % 360,
                   missing_chroma ? nil : saturation,
                   missing_lightness ? nil : lightness * 100,
                   alpha
@@ -67,7 +67,7 @@ def convert(dest, red, green, blue, alpha,
                 Color.send(
                   :for_space_internal,
                   dest,
-                  missing_hue || FuzzyMath.greater_than_or_equals(whiteness + blackness, 100) ? nil : hue % 360,
+                  missing_hue || FuzzyMath.greater_than_or_equals?(whiteness + blackness, 100) ? nil : hue % 360,
                   whiteness,
                   blackness,
                   alpha
diff --git a/lib/sass/value/color/space/utils.rb b/lib/sass/value/color/space/utils.rb
@@ -62,7 +62,7 @@ def srgb_and_display_p3_from_linear(channel)
           def lab_to_lch(dest, lightness, a, b, alpha, # rubocop:disable Naming/MethodParameterName
                          missing_chroma: false, missing_hue: false)
             chroma = Math.sqrt(((a.nil? ? 0 : a)**2) + ((b.nil? ? 0 : b)**2))
-            hue = if missing_hue || FuzzyMath.equals(chroma, 0)
+            hue = if missing_hue || FuzzyMath.equals?(chroma, 0)
                     nil
                   else
                     Math.atan2(b.nil? ? 0 : b, a.nil? ? 0 : a) * 180 / Math::PI
diff --git a/lib/sass/value/fuzzy_math.rb b/lib/sass/value/fuzzy_math.rb
@@ -12,15 +12,15 @@ module FuzzyMath
 
       module_function
 
-      def equals(number1, number2)
+      def equals?(number1, number2)
         return true if number1 == number2
 
         (number1 - number2).abs <= EPSILON &&
           (number1 * INVERSE_EPSILON).round ==
             (number2 * INVERSE_EPSILON).round
       end
 
-      def equals_nilable(number1, number2)
+      def equals_nilable?(number1, number2)
         return true if number1 == number2
         return false if number1.nil? || number2.nil?
 
@@ -29,35 +29,35 @@ def equals_nilable(number1, number2)
             (number2 * INVERSE_EPSILON).round
       end
 
-      def less_than(number1, number2)
-        number1 < number2 && !equals(number1, number2)
+      def less_than?(number1, number2)
+        number1 < number2 && !equals?(number1, number2)
       end
 
-      def less_than_or_equals(number1, number2)
-        number1 < number2 || equals(number1, number2)
+      def less_than_or_equals?(number1, number2)
+        number1 < number2 || equals?(number1, number2)
       end
 
-      def greater_than(number1, number2)
-        number1 > number2 && !equals(number1, number2)
+      def greater_than?(number1, number2)
+        number1 > number2 && !equals?(number1, number2)
       end
 
-      def greater_than_or_equals(number1, number2)
-        number1 > number2 || equals(number1, number2)
+      def greater_than_or_equals?(number1, number2)
+        number1 > number2 || equals?(number1, number2)
       end
 
       def integer?(number)
         return false unless number.finite?
 
-        equals(number, number.round)
+        equals?(number, number.round)
       end
 
       def to_i(number)
         integer?(number) ? number.round : nil
       end
 
       def between(number, min, max)
-        return min if equals(number, min)
-        return max if equals(number, max)
+        return min if equals?(number, min)
+        return max if equals?(number, max)
         return number if number > min && number < max
 
         nil
diff --git a/lib/sass/value/null.rb b/lib/sass/value/null.rb
@@ -31,7 +31,7 @@ def hash
       end
 
       # @return [::Boolean]
-      def to_bool
+      def to_bool # rubocop:disable Naming/PredicateMethod
         false
       end
 
diff --git a/lib/sass/value/number.rb b/lib/sass/value/number.rb
@@ -80,14 +80,14 @@ def ==(other)
         return false if numerator_units.length != other.numerator_units.length ||
                         denominator_units.length != other.denominator_units.length
 
-        return FuzzyMath.equals(value, other.value) if unitless?
+        return FuzzyMath.equals?(value, other.value) if unitless?
 
         if Unit.canonicalize_units(numerator_units) != Unit.canonicalize_units(other.numerator_units) &&
            Unit.canonicalize_units(denominator_units) != Unit.canonicalize_units(other.denominator_units)
           return false
         end
 
-        FuzzyMath.equals(
+        FuzzyMath.equals?(
           value *
           Unit.canonical_multiplier(numerator_units) /
           Unit.canonical_multiplier(denominator_units),
@@ -306,7 +306,7 @@ def coerce_or_convert_value(new_numerator_units, new_denominator_units,
 
           unit_length = new_numerator_units.length + new_denominator_units.length
           units = unit_string(new_numerator_units, new_denominator_units)
-          Sass::ScriptError.new("Expected #{self} to have unit#{unit_length > 1 ? 's' : ''} #{units}", name)
+          Sass::ScriptError.new("Expected #{self} to have unit#{'s' if unit_length > 1} #{units}", name)
         }
 
         result = value
diff --git a/spec/sandbox.rb b/spec/sandbox.rb
@@ -45,7 +45,7 @@ def chdir(...)
     private
 
     def file_uri(path)
-      URI::File.build([nil, "#{path.start_with?('/') ? '' : '/'}#{URI::DEFAULT_PARSER.escape(path)}"])
+      URI::File.build([nil, "#{'/' unless path.start_with?('/')}#{URI::DEFAULT_PARSER.escape(path)}"])
     end
   end
 end
