scss_lint 0.43.1 → 0.43.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 13e3b02dea61e5e42cc983459ef0269a92fd4852
-  data.tar.gz: 6c8b99c3ed5d6987e62fbfa386ba865a88fd3a6c
+  metadata.gz: 1e6187740250928694fb6a0deda93e0d3c38019a
+  data.tar.gz: 5e779311f77ca29e8f2e1ddbafc1ad4c41650351
 SHA512:
-  metadata.gz: 4fc569e9526be141eddbdaec286d960501c8717b82f09c944cf07721d3681144332706c4236fbbd280e9ea43a402ab66d1b34add10bbf8bfa9f23ad580da7ffb
-  data.tar.gz: 9d900a09921c023a4460dcf37428b5db617858bb35701df3f71aca9267c7e4fddfcb5cf03b3c89dc1f68e304cb533ef74daa25d593dc33830a5b7792c363c857
+  metadata.gz: fe7b37e6ad330637fe0a6266fd5695fa2c8ae84e47426b0f77b28c599694e1cc81df334308380f447ccffa13a668264e665ef2936e12e3741f27c73b68d1f350
+  data.tar.gz: b9773b48c133d3c02e91446a150b595a95941c52301957548a13a6e9e3d1f21148c63dff87d0773e9d72279af13127818c1f105dd0261efa57f344176847aa64

data/lib/scss_lint/engine.rb

@@ -47,7 +47,7 @@ module SCSSLint
     #   given
     def build_from_file(options)
       @filename = options[:path]
-      @contents = options[:file] ? file.read : File.read(@filename)
+      @contents = options[:file] ? options[:file].read : File.read(@filename)
       @engine = Sass::Engine.new(@contents, ENGINE_OPTIONS.merge(filename: @filename))
     end
 

data/lib/scss_lint/linter/chained_classes.rb

@@ -4,9 +4,11 @@ module SCSSLint
     include LinterRegistry
 
     def visit_sequence(sequence)
-      sequence.members.each do |simple_sequence|
-        next unless chained_class?(simple_sequence)
-        add_lint(simple_sequence.line,
+      line_offset = 0
+      sequence.members.each do |member|
+        line_offset += 1 if member =~ /\n/
+        next unless chained_class?(member)
+        add_lint(member.line + line_offset,
                  'Prefer using a distinct class over chained classes ' \
                  '(e.g. .new-class over .foo.bar')
       end
@@ -15,6 +17,7 @@ module SCSSLint
   private
 
     def chained_class?(simple_sequence)
+      return unless simple_sequence.is_a?(Sass::Selector::SimpleSequence)
       simple_sequence.members.count { |member| member.is_a?(Sass::Selector::Class) } >= 2
     end
   end

data/lib/scss_lint/version.rb

@@ -1,4 +1,4 @@
 # Defines the gem version.
 module SCSSLint
-  VERSION = '0.43.1'
+  VERSION = '0.43.2'
 end

data/spec/scss_lint/linter/chained_classes_spec.rb

@@ -42,4 +42,13 @@ describe SCSSLint::Linter::ChainedClasses do
 
     it { should report_lint line: 1 }
   end
+
+  context 'with a chained class in a multiline comma sequence' do
+    let(:scss) { <<-SCSS }
+      .one,
+      .two.three {}
+    SCSS
+
+    it { should report_lint line: 2 }
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: scss_lint
 version: !ruby/object:Gem::Version
-  version: 0.43.1
+  version: 0.43.2
 platform: ruby
 authors:
 - Brigade Engineering