scss_lint 0.45.0 → 0.46.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3c80256e0f694ade8ef5c0651dfc3cd95d1ed76a
-  data.tar.gz: ea61d9083a223c299df36c0bc827e92716b941e5
+  metadata.gz: 9c199f69c02fbbb36ce36e564adaa78528fd5b49
+  data.tar.gz: 7372af4e048f1935dc730dfffacf89bcc4a1faed
 SHA512:
-  metadata.gz: bdd1c5afb4507f544ec533ee8387506edcf1ad30e8466b0d01a0faaec2db376c4413fbd7b822fac687ba6c7ffd9672ad8103fdf2796a8521c6ae08815511269d
-  data.tar.gz: 22a40913e53ebf7e82b181aca67af574a26d5a00ce10b7b090c77f7ba1756e084b038d634b0ff4dc244bd0561922a02f2a780b0f073aa4419e93a7638fea54b9
+  metadata.gz: dad57b1b932447052533abef8b58f2bd0288badf398b2eb946ed7bcfa8dc225e2615e7b90fce017092d5647c1057600b1e7edde12f5fd50781c358530b84e88e
+  data.tar.gz: 3581b2b0fb3d039346241eec26c889083fca38b0d3f1021052e94107bc0b766e636dd370af2c0e9c92fad70a9df7a5e3494c6f50f4d00005427ceb3b22fab54d

data/config/default.yml

@@ -116,6 +116,10 @@ linters:
   PlaceholderInExtend:
     enabled: true
 
+  PrivateNamingConvention:
+    enabled: false
+    prefix: _
+
   PropertyCount:
     enabled: false
     include_nested: false

data/lib/scss_lint/linter/duplicate_property.rb

@@ -3,13 +3,22 @@ module SCSSLint
   class Linter::DuplicateProperty < Linter
     include LinterRegistry
 
+    def visit_root(_node)
+      @ignore_consecutive = config['ignore_consecutive']
+      yield
+    end
+
     def check_properties(node)
       static_properties(node).each_with_object({}) do |prop, prop_names|
         prop_key = property_key(prop)
 
         if existing_prop = prop_names[prop_key]
-          add_lint(prop, "Property `#{existing_prop.name.join}` already "\
-                         "defined on line #{existing_prop.line}")
+          if existing_prop.line < prop.line - 1 || !ignore_consecutive_of?(prop)
+            add_lint(prop, "Property `#{existing_prop.name.join}` already "\
+                           "defined on line #{existing_prop.line}")
+          else
+            prop_names[prop_key] = prop
+          end
         else
           prop_names[prop_key] = prop
         end
@@ -54,5 +63,21 @@ module SCSSLint
         prop.value.to_s
       end
     end
+
+    def ignore_consecutive_of?(prop)
+      case @ignore_consecutive
+      when true
+        return true
+      when false
+        return false
+      when nil
+        return false
+      when Array
+        return @ignore_consecutive.include?(prop.name.join)
+      else
+        raise SCSSLint::Exceptions::LinterError,
+              "#{@ignore_consecutive.inspect} is not a valid value for ignore_consecutive."
+      end
+    end
   end
 end

data/lib/scss_lint/linter/private_naming_convention.rb

@@ -0,0 +1,157 @@
+module SCSSLint
+  # Verifies that variables, functions, and mixins that follow the private
+  # naming convention are defined and used within the same file.
+  class Linter::PrivateNamingConvention < Linter # rubocop:disable ClassLength
+    include LinterRegistry
+
+    DEFINITIONS = {
+      Sass::Tree::FunctionNode => {
+        defines: Sass::Script::Tree::Funcall,
+        human_name: 'function',
+      },
+      Sass::Tree::MixinDefNode => {
+        defines: Sass::Tree::MixinNode,
+        human_name: 'mixin',
+      },
+      Sass::Tree::VariableNode => {
+        defines: Sass::Script::Tree::Variable,
+        human_name: 'variable',
+      },
+    }.freeze
+
+    HUMAN_NODE_NAMES = Hash[DEFINITIONS.map { |k, v| [k, v[:human_name]] }].freeze
+    DEFINED_BYS = Hash[DEFINITIONS.map { |k, v| [v[:defines], k] }].freeze
+
+    def visit_root(node)
+      # Register all top-level function, mixin, and variable definitions.
+      node.children.each_with_object([]) do |child_node|
+        if DEFINITIONS.key?(child_node.class)
+          register_node child_node
+        else
+          yield
+        end
+      end
+
+      # After we have visited everything, we want to see if any private things
+      # were defined but not used.
+      after_visit_all
+    end
+
+    def visit_script_funcall(node)
+      check_privacy(node)
+      yield # Continue linting any arguments of this function call
+    end
+
+    def visit_mixin(node)
+      check_privacy(node)
+      yield # Continue into content block of this mixin's block
+    end
+
+    def visit_script_variable(node)
+      check_privacy(node)
+    end
+
+  private
+
+    def register_node(node, node_text = node.name)
+      return unless private?(node)
+
+      @private_definitions ||= {}
+      @private_definitions[node.class] ||= {}
+
+      @private_definitions[node.class][node_text] = {
+        node: node,
+        times_used: 0,
+      }
+    end
+
+    def check_privacy(node, node_text = node.name)
+      return unless private?(node)
+
+      defined_by_class = defined_by(node)
+
+      # Look at top-level private definitions
+      if @private_definitions &&
+          @private_definitions[defined_by_class] &&
+          @private_definitions[defined_by_class][node_text]
+        @private_definitions[defined_by_class][node_text][:times_used] += 1
+        return
+      end
+
+      # We did not find a top-level private definition, so let's traverse up the
+      # tree, looking for private definitions of this node that are scoped.
+      looking_for = {
+        node: node,
+        defined_by: defined_by_class,
+        location: location_from_range(node.source_range),
+      }
+      return if node_defined_earlier_in_branch?(node.node_parent, looking_for)
+
+      node_type = humanize_node_class(node)
+      add_lint(
+        node,
+        "Private #{node_type} #{node_text} must be defined in the same file it is used"
+      )
+    end
+
+    def node_defined_earlier_in_branch?(node_to_look_in, looking_for)
+      # Look at all of the children of this node and return true if we find a
+      # defining node that matches in name and type.
+      node_to_look_in.children.each_with_object([]) do |child_node|
+        break unless before?(child_node, looking_for[:location])
+        next unless child_node.class == looking_for[:defined_by]
+        next unless child_node.name == looking_for[:node].name
+
+        return true # We found a match, so we are done
+      end
+
+      # We are at the top of the branch and don't want to check the root branch,
+      # since that is handled elsewhere, which means that we did not find a
+      # match.
+      return false unless node_to_look_in.node_parent.node_parent
+
+      # We did not find a match yet, and haven't reached the top of the branch,
+      # so recurse.
+      if node_to_look_in.node_parent
+        node_defined_earlier_in_branch?(node_to_look_in.node_parent, looking_for)
+      end
+    end
+
+    def private?(node)
+      node.name.start_with?(config['prefix'])
+    end
+
+    def before?(node, before_location)
+      location = location_from_range(node.source_range)
+      return true if location.line < before_location.line
+      if location.line == before_location.line &&
+          location.column < before_location.column
+        return true
+      end
+      false
+    end
+
+    def after_visit_all
+      return unless @private_definitions
+
+      @private_definitions.each do |_, nodes|
+        nodes.each do |node_text, node_info|
+          next if node_info[:times_used] > 0
+          node_type = humanize_node_class(node_info[:node])
+          add_lint(
+            node_info[:node],
+            "Private #{node_type} #{node_text} must be used in the same file it is defined"
+          )
+        end
+      end
+    end
+
+    def humanize_node_class(node)
+      HUMAN_NODE_NAMES[node.class]
+    end
+
+    def defined_by(node)
+      DEFINED_BYS[node.class]
+    end
+  end
+end

data/lib/scss_lint/reporter/tap_reporter.rb

@@ -0,0 +1,123 @@
+module SCSSLint
+  # Reports in TAP format.
+  # http://testanything.org/
+  class Reporter::TAPReporter < Reporter
+    TAP_VERSION = 'TAP version 13'.freeze
+
+    def report_lints
+      output = [TAP_VERSION, format_plan(files, lints)]
+      return format_output(output) unless files.any?
+
+      output.concat(format_files(files, lints))
+      format_output(output)
+    end
+
+  private
+
+    # @param files [Array<String>]
+    # @param lints [Array<SCSSLint::Lint>]
+    # @return [String]
+    def format_plan(files, lints)
+      files_with_lints = lints.map(&:filename).uniq
+      extra_lines = lints.count - files_with_lints.count
+      comment = files.count == 0 ? ' # No files to lint' : ''
+      "1..#{files.count + extra_lines}#{comment}"
+    end
+
+    # @param files [Array<String>]
+    # @param lints [Array<SCSSLint::Lint>]
+    # @return [Array<String>] one item per ok file or not ok lint
+    def format_files(files, lints)
+      unless lints.any?
+        # There are no lints, so we can take a shortcut and just output an ok
+        # test line for every file.
+        return files.map.with_index do |filename, index|
+          format_ok(filename, index + 1)
+        end
+      end
+
+      # There are lints, so we need to go through each file, find the lints
+      # for the file, and add them to the output.
+
+      # Since we'll be looking up lints by filename for each filename, we want
+      # to make a first pass to group all of the lints by filename to make
+      # lookup fast.
+      grouped_lints = group_lints_by_filename(lints)
+
+      test_number = 1
+      files.map do |filename|
+        if grouped_lints.key?(filename)
+          # This file has lints, so we want to generate a "not ok" test line for
+          # each failing lint.
+          grouped_lints[filename].map do |lint|
+            formatted = format_not_ok(lint, test_number)
+            test_number += 1
+            formatted
+          end
+        else
+          formatted = format_ok(filename, test_number)
+          test_number += 1
+          [formatted]
+        end
+      end.flatten
+    end
+
+    # @param lints [Array<SCSSLint::Lint>]
+    # @return [Hash] keyed by filename, values are arrays of lints
+    def group_lints_by_filename(lints)
+      grouped_lints = {}
+      lints.each do |lint|
+        grouped_lints[lint.filename] ||= []
+        grouped_lints[lint.filename] << lint
+      end
+      grouped_lints
+    end
+
+    # @param filename [String]
+    # @param test_number [Number]
+    # @return [String]
+    def format_ok(filename, test_number)
+      "ok #{test_number} - #{filename}"
+    end
+
+    # @param lint [SCSSLint::Lint]
+    # @param test_number [Number]
+    # @return [String]
+    def format_not_ok(lint, test_number)
+      location = lint.location
+      test_line_description = "#{lint.filename}:#{location.line}:#{location.column}"
+      test_line_description += " #{lint.linter.name}" if lint.linter
+
+      <<-EOS.strip
+not ok #{test_number} - #{test_line_description}
+  ---
+  message: #{lint.description}
+  severity: #{lint.severity}
+  data:
+    file: #{lint.filename}
+    line: #{lint.location.line}
+    column: #{lint.location.column}
+  ---
+      EOS
+    end
+
+    # @param output [Array<String>]
+    # @return [String]
+    def format_output(output)
+      output.join("\n") + "\n"
+    end
+
+    def location(lint)
+      "#{log.cyan(lint.filename)}:#{log.magenta(lint.location.line.to_s)}"
+    end
+
+    def type(lint)
+      lint.error? ? log.red('[E]') : log.yellow('[W]')
+    end
+
+    def message(lint)
+      linter_name = log.green("#{lint.linter.name}: ") if lint.linter
+      "#{linter_name}#{lint.description}"
+    end
+  end
+end

data/lib/scss_lint/version.rb

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

data/spec/scss_lint/linter/duplicate_property_spec.rb

@@ -186,4 +186,59 @@ describe SCSSLint::Linter::DuplicateProperty do
 
     it { should report_lint line: 4 }
   end
+
+  context 'when consecutive duplicate properties are allowed' do
+    let(:linter_config) { { 'ignore_consecutive' => true } }
+
+    context 'when rule set contains consecutive duplicates' do
+      let(:scss) { <<-SCSS }
+        p {
+          background-color: #fff;
+          background-color: rgba(255, 255, 255, 0.7);
+          background-color: a-third-thing;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when rule set contains non-consecutive duplicates' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin: 0;
+          padding: 0;
+          margin: 1em;
+        }
+      SCSS
+
+      it { should report_lint line: 4 }
+    end
+  end
+
+  context 'when specific consecutive duplicate properties are allowed' do
+    let(:linter_config) { { 'ignore_consecutive' => ['background-color', 'transition'] } }
+
+    context 'when rule set contains consecutive duplicates in whitelist' do
+      let(:scss) { <<-SCSS }
+        p {
+          background-color: #fff;
+          background-color: rgba(255, 255, 255, 0.7);
+          background-color: a-third-thing;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'when rule set contains consecutive duplicates not in whitelist' do
+      let(:scss) { <<-SCSS }
+        p {
+          margin: 0;
+          margin: 5px;
+        }
+      SCSS
+
+      it { should report_lint line: 3 }
+    end
+  end
 end

data/spec/scss_lint/linter/private_naming_convention_spec.rb

@@ -0,0 +1,445 @@
+require 'spec_helper'
+
+describe SCSSLint::Linter::PrivateNamingConvention do
+  context 'when a private variable' do
+    context 'is not used in the same file it is defined' do
+      let(:scss) { <<-SCSS }
+        $_foo: red;
+      SCSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'is used but not defined in the same file' do
+      let(:scss) { <<-SCSS }
+        p {
+          color: $_foo;
+          background: rgba($_foo, 0);
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+      it { should report_lint line: 3 }
+    end
+
+    context 'is used but has not been defined quite yet' do
+      let(:scss) { <<-SCSS }
+        p {
+          color: $_foo;
+          background: rgba($_foo, 0);
+        }
+
+        $_foo: red;
+      SCSS
+
+      it { should report_lint line: 2 }
+      it { should report_lint line: 3 }
+    end
+
+    context 'is defined and used in the same file' do
+      let(:scss) { <<-SCSS }
+        $_foo: red;
+
+        p {
+          color: $_foo;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'is defined and used in the same file in a function' do
+      let(:scss) { <<-SCSS }
+        $_foo: red;
+
+        p {
+          color: rgba($_foo, 0);
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'is defined within a selector and not used' do
+      let(:scss) { <<-SCSS }
+        p {
+          $_foo: red;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'is defined within a selector and used' do
+      let(:scss) { <<-SCSS }
+        p {
+          $_foo: red;
+          color: $_foo;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'is defined within a selector and used in a nested selector' do
+      let(:scss) { <<-SCSS }
+        p {
+          $_foo: red;
+
+          a {
+            color: $_foo;
+          }
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'is defined within a selector but too late' do
+      let(:scss) { <<-SCSS }
+        p {
+          color: $_foo;
+          $_foo: red;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'is defined within a selector but after the nested selector it is used in' do
+      let(:scss) { <<-SCSS }
+        p {
+          a {
+            color: $_foo;
+          }
+
+          $_foo: red;
+        }
+      SCSS
+
+      it { should report_lint line: 3 }
+    end
+
+    context 'is defined in a different selector than it is used in' do
+      let(:scss) { <<-SCSS }
+        p {
+          $_foo: red;
+        }
+
+        a {
+          color: $_foo;
+        }
+      SCSS
+
+      it { should report_lint line: 6 }
+    end
+  end
+
+  context 'when a public variable' do
+    context 'is used but not defined' do
+      let(:scss) { <<-SCSS }
+        p {
+          color: $foo;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'is defined but not used' do
+      let(:scss) { <<-SCSS }
+        $foo: red;
+      SCSS
+
+      it { should_not report_lint }
+    end
+  end
+
+  context 'when a private mixin' do
+    context 'is not used in the same file it is defined' do
+      let(:scss) { <<-SCSS }
+        @mixin _foo {
+          color: red;
+        }
+      SCSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'is used but not defined in the same file' do
+      let(:scss) { <<-SCSS }
+        p {
+          @include _foo;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'is used but has not been defined quite yet' do
+      let(:scss) { <<-SCSS }
+        p {
+          @include _foo;
+        }
+
+        @mixin _foo {
+          color: red;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'is defined within a selector and not used' do
+      let(:scss) { <<-SCSS }
+        p {
+          @mixin _foo {
+            color: red;
+          }
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'is defined within a selector and used within that selector' do
+      let(:scss) { <<-SCSS }
+        p {
+          @mixin _foo {
+            color: red;
+          }
+
+          @include _foo;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'is defined within a selector and used within a nested selector' do
+      let(:scss) { <<-SCSS }
+        p {
+          @mixin _foo {
+            color: red;
+          }
+
+          a {
+            @include _foo;
+          }
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'is defined within a selector and used within that selector too early' do
+      let(:scss) { <<-SCSS }
+        p {
+          @include _foo;
+
+          @mixin _foo {
+            color: red;
+          }
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'is defined within a selector and used within a nested selector too early' do
+      let(:scss) { <<-SCSS }
+        p {
+          a {
+            @include _foo;
+          }
+
+          @mixin _foo {
+            color: red;
+          }
+        }
+      SCSS
+
+      it { should report_lint line: 3 }
+    end
+
+    context 'is defined and used in the same file' do
+      let(:scss) { <<-SCSS }
+        @mixin _foo {
+          color: red;
+        }
+
+        p {
+          @include _foo;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+  end
+
+  context 'when a public mixin' do
+    context 'is used but not defined' do
+      let(:scss) { <<-SCSS }
+        p {
+          @include foo;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'is defined but not used' do
+      let(:scss) { <<-SCSS }
+        @mixin foo {
+          color: red;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+  end
+
+  describe 'when a private function' do
+    context 'is not used in the same file it is defined' do
+      let(:scss) { <<-SCSS }
+        @function _foo() {
+          @return red;
+        }
+      SCSS
+
+      it { should report_lint line: 1 }
+    end
+
+    context 'is used but not defined in the same file' do
+      let(:scss) { <<-SCSS }
+        p {
+          color: _foo();
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'is used but has not been defined quite yet' do
+      let(:scss) { <<-SCSS }
+        p {
+          color: _foo();
+        }
+
+        @function _foo() {
+          @return red;
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'is defined within a selector and not used' do
+      let(:scss) { <<-SCSS }
+        p {
+          @function _foo() {
+            @return red;
+          }
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'is defined within a selector and used within the same selector' do
+      let(:scss) { <<-SCSS }
+        p {
+          @function _foo() {
+            @return red;
+          }
+
+          color: _foo();
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'is defined within a selector and used within a nested selector' do
+      let(:scss) { <<-SCSS }
+        p {
+          @function _foo() {
+            @return red;
+          }
+
+          a {
+            color: _foo();
+          }
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'is defined within a selector and used within the same selector too early' do
+      let(:scss) { <<-SCSS }
+        p {
+          color: _foo();
+
+          @function _foo() {
+            @return red;
+          }
+        }
+      SCSS
+
+      it { should report_lint line: 2 }
+    end
+
+    context 'is defined within a selector and used within a nested too early' do
+      let(:scss) { <<-SCSS }
+        p {
+          a {
+            color: _foo();
+          }
+
+          @function _foo() {
+            @return red;
+          }
+        }
+      SCSS
+
+      it { should report_lint line: 3 }
+    end
+
+    context 'is defined and used in the same file' do
+      let(:scss) { <<-SCSS }
+        @function _foo() {
+          @return red;
+        }
+
+        p {
+          color: _foo();
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+  end
+
+  context 'when a public function' do
+    context 'is used but not defined' do
+      let(:scss) { <<-SCSS }
+        p {
+          color: foo();
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+
+    context 'is defined but not used' do
+      let(:scss) { <<-SCSS }
+        @function foo() {
+          @return red;
+        }
+      SCSS
+
+      it { should_not report_lint }
+    end
+  end
+end

data/spec/scss_lint/reporter/tap_reporter_spec.rb

@@ -0,0 +1,97 @@
+require 'spec_helper'
+
+describe SCSSLint::Reporter::TAPReporter do
+  let(:logger) { SCSSLint::Logger.new($stdout) }
+  subject { described_class.new(lints, filenames, logger) }
+
+  describe '#report_lints' do
+    context 'when there are no files' do
+      let(:filenames) { [] }
+      let(:lints) { [] }
+
+      it 'returns the TAP version, plan, and explanation' do
+        subject.report_lints.should == "TAP version 13\n1..0 # No files to lint\n"
+      end
+    end
+
+    context 'when there are files but no lints' do
+      let(:filenames) { ['file.scss', 'another-file.scss'] }
+      let(:lints) { [] }
+
+      it 'returns the TAP version, plan, and ok test lines' do
+        subject.report_lints.should eq(<<-EOS)
+TAP version 13
+1..2
+ok 1 - file.scss
+ok 2 - another-file.scss
+        EOS
+      end
+    end
+
+    context 'when there are some lints' do
+      let(:filenames) { %w[ok1.scss not-ok1.scss not-ok2.scss ok2.scss] }
+
+      let(:lints) do
+        [
+          SCSSLint::Lint.new(
+            SCSSLint::Linter::PrivateNamingConvention,
+            filenames[1],
+            SCSSLint::Location.new(123, 10, 8),
+            'Description of lint 1',
+            :warning
+          ),
+          SCSSLint::Lint.new(
+            SCSSLint::Linter::PrivateNamingConvention,
+            filenames[2],
+            SCSSLint::Location.new(20, 2, 6),
+            'Description of lint 2',
+            :error
+          ),
+          SCSSLint::Lint.new(
+            SCSSLint::Linter::PrivateNamingConvention,
+            filenames[2],
+            SCSSLint::Location.new(21, 3, 4),
+            'Description of lint 3',
+            :warning
+          ),
+        ]
+      end
+
+      it 'returns the TAP version, plan, and correct test lines' do
+        subject.report_lints.should eq(<<-EOS)
+TAP version 13
+1..5
+ok 1 - ok1.scss
+not ok 2 - not-ok1.scss:123:10 SCSSLint::Linter::PrivateNamingConvention
+  ---
+  message: Description of lint 1
+  severity: warning
+  data:
+    file: not-ok1.scss
+    line: 123
+    column: 10
+  ---
+not ok 3 - not-ok2.scss:20:2 SCSSLint::Linter::PrivateNamingConvention
+  ---
+  message: Description of lint 2
+  severity: error
+  data:
+    file: not-ok2.scss
+    line: 20
+    column: 2
+  ---
+not ok 4 - not-ok2.scss:21:3 SCSSLint::Linter::PrivateNamingConvention
+  ---
+  message: Description of lint 3
+  severity: warning
+  data:
+    file: not-ok2.scss
+    line: 21
+    column: 3
+  ---
+ok 5 - ok2.scss
+        EOS
+      end
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: scss_lint
 version: !ruby/object:Gem::Version
-  version: 0.45.0
+  version: 0.46.0
 platform: ruby
 authors:
 - Brigade Engineering
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-17 00:00:00.000000000 Z
+date: 2016-02-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -118,6 +118,7 @@ files:
 - lib/scss_lint/linter/name_format.rb
 - lib/scss_lint/linter/nesting_depth.rb
 - lib/scss_lint/linter/placeholder_in_extend.rb
+- lib/scss_lint/linter/private_naming_convention.rb
 - lib/scss_lint/linter/property_count.rb
 - lib/scss_lint/linter/property_sort_order.rb
 - lib/scss_lint/linter/property_spelling.rb
@@ -163,6 +164,7 @@ files:
 - lib/scss_lint/reporter/default_reporter.rb
 - lib/scss_lint/reporter/files_reporter.rb
 - lib/scss_lint/reporter/json_reporter.rb
+- lib/scss_lint/reporter/tap_reporter.rb
 - lib/scss_lint/runner.rb
 - lib/scss_lint/sass/script.rb
 - lib/scss_lint/sass/tree.rb
@@ -203,6 +205,7 @@ files:
 - spec/scss_lint/linter/name_format_spec.rb
 - spec/scss_lint/linter/nesting_depth_spec.rb
 - spec/scss_lint/linter/placeholder_in_extend_spec.rb
+- spec/scss_lint/linter/private_naming_convention_spec.rb
 - spec/scss_lint/linter/property_count_spec.rb
 - spec/scss_lint/linter/property_sort_order_spec.rb
 - spec/scss_lint/linter/property_spelling_spec.rb
@@ -249,6 +252,7 @@ files:
 - spec/scss_lint/reporter/default_reporter_spec.rb
 - spec/scss_lint/reporter/files_reporter_spec.rb
 - spec/scss_lint/reporter/json_reporter_spec.rb
+- spec/scss_lint/reporter/tap_reporter_spec.rb
 - spec/scss_lint/reporter_spec.rb
 - spec/scss_lint/runner_spec.rb
 - spec/scss_lint/selector_visitor_spec.rb
@@ -314,6 +318,7 @@ test_files:
 - spec/scss_lint/linter/name_format_spec.rb
 - spec/scss_lint/linter/nesting_depth_spec.rb
 - spec/scss_lint/linter/placeholder_in_extend_spec.rb
+- spec/scss_lint/linter/private_naming_convention_spec.rb
 - spec/scss_lint/linter/property_count_spec.rb
 - spec/scss_lint/linter/property_sort_order_spec.rb
 - spec/scss_lint/linter/property_spelling_spec.rb
@@ -360,6 +365,7 @@ test_files:
 - spec/scss_lint/reporter/default_reporter_spec.rb
 - spec/scss_lint/reporter/files_reporter_spec.rb
 - spec/scss_lint/reporter/json_reporter_spec.rb
+- spec/scss_lint/reporter/tap_reporter_spec.rb
 - spec/scss_lint/reporter_spec.rb
 - spec/scss_lint/runner_spec.rb
 - spec/scss_lint/selector_visitor_spec.rb