scss_lint 0.53.0 → 0.54.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 28bb5edd241b1790563ffb57972f89f8a169f9af
-  data.tar.gz: c1a9b234a92f4729a7ec3353fd7e7c7d03fff662
+  metadata.gz: aff9c12976312f5790dca3b5423ae1413cfbd1ec
+  data.tar.gz: 138c24adcf244fa81f594e3c2a1b82003da97782
 SHA512:
-  metadata.gz: d29131bc10b67067ad84e4ebc5aac6a8857a017ae963b26e54804338d50bad871756c373e7588929e1c0e5a43334b89d09f44788ace9ad9a79ef3586d4ac4229
-  data.tar.gz: e0aa4f037e09dde3fa7487dcab53802d8cd3f481ac083b19f201500d059d2cdf90c1c4d2e92d33c5f075b56849bf93f9913ed0191ade5d8e8135dd804e611954
+  metadata.gz: 5f9f93bcf637bd292942415c4570a75ccecc595819e1a5890e8846c38e9f23a16cfa4a93cb73a292b8e87f824a86fb95b6334f868cc05a80b2b610e3096a86dd
+  data.tar.gz: 6a65980f9c12a3c7bdd0b085f189fcb6b0fbc7e5063acca043de4960154e825d4ebe7bcdb44837903b4571a907f108864b3dc1065d244e2031da96991e8eed02

data/data/properties.txt

@@ -208,13 +208,22 @@ grid-auto-flow
 grid-auto-rows
 grid-before
 grid-column
+grid-column-end
+grid-column-gap
+grid-column-start
 grid-columns
 grid-end
+grid-gap
 grid-row
+grid-row-end
+grid-row-gap
+grid-row-start
 grid-rows
 grid-start
 grid-template
 grid-template-areas
+grid-template-columns
+grid-template-rows
 hanging-punctuation
 height
 highlight

data/lib/scss_lint/linter.rb

@@ -75,6 +75,10 @@ module SCSSLint
                  line_source.length - range.start_pos.offset + 1
                end
 
+      # Workaround for https://github.com/brigade/scss-lint/issues/887 to acount for
+      # https://github.com/sass/sass/issues/2284.
+      length = 1 if length < 1
+
       Location.new(range.start_pos.line, range.start_pos.offset, length)
     end
 

data/lib/scss_lint/linter/indentation.rb

@@ -97,8 +97,8 @@ module SCSSLint
     end
 
     def visit_import(node)
-      prev = previous_node(node)
-      return if prev.is_a?(Sass::Tree::ImportNode) && source_from_range(prev.source_range) =~ /,$/
+      previous_node(node)
+      return unless engine.lines[node.line - 1] =~ /@import/
       check_indentation(node)
     end
 

data/lib/scss_lint/linter/name_format.rb

@@ -10,6 +10,8 @@ module SCSSLint
       translateX translateY translateZ
       linear-gradient repeating-linear-gradient
       radial-gradient repeating-radial-gradient
+      cubic-bezier
+      fit-content
     ].to_set.freeze
 
     SCSS_FUNCTION_WHITELIST = %w[

data/lib/scss_lint/linter/space_after_comment.rb

@@ -58,6 +58,9 @@ module SCSSLint
     def whitespace_after_comment(source, offset)
       whitespace = 0
 
+      # Allow for comments that start with `/// `.
+      offset += 1 if source[offset] == '/'
+
       while [' ', "\t"].include? source[offset]
         whitespace += 1
         offset += 1

data/lib/scss_lint/linter/space_after_property_colon.rb

@@ -47,7 +47,7 @@ module SCSSLint
     end
 
     def check_for_one_space_or_newline(node, whitespace)
-      return if whitespace == [' '] || whitespace == ["\n"]
+      return if [[' '], ["\n"]].include?(whitespace)
       return if whitespace[0] == "\n" && whitespace[1..-1].uniq == [' ']
       add_lint(node, 'Colon after property should be followed by one space or a newline')
     end

data/lib/scss_lint/linter/space_after_variable_colon.rb

@@ -37,7 +37,7 @@ module SCSSLint
     end
 
     def check_for_one_space_or_newline(node, whitespace)
-      return if whitespace == [' '] || whitespace == ["\n"]
+      return if [[' '], ["\n"]].include?(whitespace)
       return if whitespace[0] == "\n" && whitespace[1..-1].uniq == [' ']
       add_lint(node, 'Colon after variable should be followed by one space or a newline')
     end
@@ -51,7 +51,7 @@ module SCSSLint
       offset = offset_to(start_pos, ':', offset) + 1
 
       # Count spaces after the colon
-      while [' ', "\t", "\n"].include? character_at(start_pos, offset)
+      while [' ', "\t", "\n"].include?(character_at(start_pos, offset))
         whitespace << character_at(start_pos, offset)
         offset += 1
       end

data/lib/scss_lint/plugins/linter_gem.rb

@@ -12,7 +12,7 @@ module SCSSLint
         require @name
         @config = plugin_config
         self
-      rescue Gem::LoadError, LoadError
+      rescue LoadError
         raise SCSSLint::Exceptions::PluginGemLoadError,
               "Unable to load linter plugin gem '#{@name}'. Try running " \
               "`gem install #{@name}`, or adding it to your Gemfile and " \

data/lib/scss_lint/version.rb

@@ -2,5 +2,5 @@
 
 # Defines the gem version.
 module SCSSLint
-  VERSION = '0.53.0'.freeze
+  VERSION = '0.54.0'.freeze
 end

data/spec/scss_lint/linter/indentation_spec.rb

@@ -238,6 +238,16 @@ describe SCSSLint::Linter::Indentation do
     it { should_not report_lint }
   end
 
+  context 'when an @import spans multiple lines and leads with a newline' do
+    let(:scss) { <<-SCSS }
+      @import
+          'foo',
+          'bar';
+    SCSS
+
+    it { should_not report_lint }
+  end
+
   context 'when tabs are preferred' do
     let(:linter_config) { { 'character' => 'tab', 'width' => 1 } }
 

data/spec/scss_lint/linter/space_after_comment_spec.rb

@@ -38,6 +38,14 @@ describe SCSSLint::Linter::SpaceAfterComment do
       it { should report_lint line: 1 }
     end
 
+    context 'when silent three-slash comment' do
+      let(:scss) { <<-SCSS }
+        /// triple slash
+      SCSS
+
+      it { should_not report_lint }
+    end
+
     context 'when inline silent comment and no space' do
       let(:scss) { <<-SCSS }
         p {

data/spec/scss_lint/runner_spec.rb

@@ -112,11 +112,11 @@ describe SCSSLint::Runner do
       end
 
       it 'has the name of the file the linter was checking' do
-        expect { subject }.to raise_error { |e| e.message.should include files.first[:path] }
+        expect { subject }.to(raise_error { |e| e.message.should include files.first[:path] })
       end
 
       it 'has the same backtrace as the original error' do
-        expect { subject }.to raise_error { |e| e.backtrace.should == backtrace }
+        expect { subject }.to(raise_error { |e| e.backtrace.should == backtrace })
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: scss_lint
 version: !ruby/object:Gem::Version
-  version: 0.53.0
+  version: 0.54.0
 platform: ruby
 authors:
 - Brigade Engineering
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-04-10 00:00:00.000000000 Z
+date: 2017-06-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -272,7 +272,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5.1
+rubygems_version: 2.6.11
 signing_key: 
 specification_version: 4
 summary: SCSS lint tool