Code style

Author: ntkme

lib/sass/embedded/host/structifier.rb

@@ -10,7 +10,7 @@ module Structifier
         module_function
 
         def to_struct(obj, *symbols)
-          return obj unless obj.is_a? Hash
+          return obj unless obj.is_a?(Hash)
 
           struct = Object.new
           symbols.each do |key|

lib/sass/value/color.rb

@@ -50,42 +50,42 @@ def initialize(red: nil,
 
       # @return [Integer]
       def red
-        hsl_to_rgb unless defined? @red
+        hsl_to_rgb unless defined?(@red)
 
         @red
       end
 
       # @return [Integer]
       def green
-        hsl_to_rgb unless defined? @green
+        hsl_to_rgb unless defined?(@green)
 
         @green
       end
 
       # @return [Integer]
       def blue
-        hsl_to_rgb unless defined? @blue
+        hsl_to_rgb unless defined?(@blue)
 
         @blue
       end
 
       # @return [Numeric]
       def hue
-        rgb_to_hsl unless defined? @hue
+        rgb_to_hsl unless defined?(@hue)
 
         @hue
       end
 
       # @return [Numeric]
       def saturation
-        rgb_to_hsl unless defined? @saturation
+        rgb_to_hsl unless defined?(@saturation)
 
         @saturation
       end
 
       # @return [Numeric]
       def lightness
-        rgb_to_hsl unless defined? @lightness
+        rgb_to_hsl unless defined?(@lightness)
 
         @lightness
       end

lib/sass/value/map.rb

@@ -30,7 +30,7 @@ def ==(other)
       # @param index [Numeric, Value]
       # @return [List<(Value, Value)>, Value]
       def at(index)
-        if index.is_a? Numeric
+        if index.is_a?(Numeric)
           index = index.floor
           index = to_a_length + index if index.negative?
           return nil if index.negative? || index >= to_a_length

lib/sass/value/number.rb

@@ -25,12 +25,12 @@ def initialize(value, unit = nil)
           denominator_units = []
         when ::Hash
           numerator_units = unit.fetch(:numerator_units, [])
-          unless numerator_units.is_a? Array
+          unless numerator_units.is_a?(Array)
             raise Sass::ScriptError, "invalid numerator_units #{numerator_units.inspect}"
           end
 
           denominator_units = unit.fetch(:denominator_units, [])
-          unless denominator_units.is_a? Array
+          unless denominator_units.is_a?(Array)
             raise Sass::ScriptError, "invalid denominator_units #{denominator_units.inspect}"
           end
         else
@@ -75,7 +75,7 @@ def initialize(value, unit = nil)
 
       # @return [::Boolean]
       def ==(other)
-        return false unless other.is_a? Sass::Value::Number
+        return false unless other.is_a?(Sass::Value::Number)
 
         return false if numerator_units.length != other.numerator_units.length ||
                         denominator_units.length != other.denominator_units.length

spec/sass_proto_spec.rb

@@ -5,7 +5,7 @@
 RSpec.describe Sass do
   def remote_eq(lhs, rhs)
     to_host_value = lambda { |value|
-      if value.is_a? Sass::Value::ArgumentList
+      if value.is_a?(Sass::Value::ArgumentList)
         value.dup
       else
         value