scss_lint-auto_correct 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fba2efff4f1df225710a78b728482a14cc8131a4
-  data.tar.gz: 12f5ab2d724a3b3127d39246d9e5441bfb130b5c
+  metadata.gz: bf50c2002cc599f3886207b12128de2325f98ad9
+  data.tar.gz: 0004de322f5480f32b27c5998a739a01d53ddbd4
 SHA512:
-  metadata.gz: c9442a72c3feefe5d8c588510f560f031a175ae7d9c5ad2ef0795bb82f9400a0e63c52b26ca872be2607ad311129dce7cb1591793ab1e29b2682afaf5ddb2f4e
-  data.tar.gz: 8fe3ac34f4de8717ca5db69bf396f0cecaa6dbfb91ebcb08601dddaf23e3c36cf92028ad263beb5e4d4761e1a4547db681be5c81b4c82528e9ca99d09fa67cc9
+  metadata.gz: 881227252dc98f9b52ec053948fa0cab2e09d6098894a51848cea90df688a257f304e4c1ed09cc18790d1e430e32c9547c838e4f17b90c7a0d00709607bdff3c
+  data.tar.gz: a292ee51dffcf338af613c4990efd162cb762afb622b004c88ce1862b12accf70f2fc7717c209f28371ede1ad56479136b0515f8deeccfa4111fafca26cc72ef

data/lib/scss_lint/auto_correct/correctors/base.rb

@@ -1,6 +1,7 @@
 module SCSSLint::AutoCorrect::Correctors
   class Base
     def self.linter_name
+      self.name.split("::").last
     end
 
     def self.short_name

data/lib/scss_lint/auto_correct/correctors/color_keyword.rb

@@ -0,0 +1,44 @@
+module SCSSLint::AutoCorrect::Correctors
+  class ColorKeyword < Base
+    def call(contents)
+      result = []
+      contents.lines.each do |line|
+        if line =~ /\s*\/\// # skip comments
+          result << line
+          next
+        end
+        mm = line.match(/(.+:)([\s\S]*)$/)
+        if mm
+          result << mm[1] + correct_values(mm[2])
+        else
+          result << line
+        end
+      end
+      result.join
+    end
+
+    def correct_values(str)
+      str.gsub(regexp) do |color|
+        self.class.data[color]
+      end
+    end
+
+    def regexp
+      @regexp ||= Regexp.new("#{self.class.data.keys.join('|')}")
+    end
+
+    def self.data
+      return @data if @data
+      colors_file = File.expand_path(File.join(File.dirname(__FILE__), "color-keywords.csv"))
+      @data = Hash[File.read(colors_file).lines.map { |l| l.gsub("\n", '').split(',') }]
+    end
+
+    def self.description
+      "Replace named colors with their hex values"
+    end
+
+    def self.priority
+      5
+    end
+  end
+end

data/lib/scss_lint/auto_correct/correctors/{format_newlines.rb → empty_line_between_blocks.rb}

@@ -1,5 +1,5 @@
 module SCSSLint::AutoCorrect::Correctors
-  class FormatNewlines < Base
+  class EmptyLineBetweenBlocks < Base
     def call(contents)
       contents.gsub(/}\n( *[\.a-zA-Z0-9=\-:&\[\]]+) {/m) do
         "}\n\n#{Regexp.last_match[1]} {"
@@ -7,7 +7,7 @@ module SCSSLint::AutoCorrect::Correctors
     end
 
     def self.description
-      ""
+      "Ensures empty lines between block defintions"
     end
   end
 end

data/lib/scss_lint/auto_correct/correctors/{shorter_versions_of_colors.rb → hex_length.rb}

@@ -1,13 +1,10 @@
 module SCSSLint::AutoCorrect::Correctors
-  class ShorterVersionsOfColors < Base
+  class HexLength < Base
     def call(contents)
+      # TODO: config['style'] == 'short'
       contents.gsub(/#(([A-Za-z0-9])\2){3}/) { |m| "##{m[1]}#{m[3]}#{m[5]}" }
     end
 
-    def self.linter_name
-      "HexLength"
-    end
-
     def self.description
       "Shortens hex color codes to 3 letters, when possible"
     end

data/lib/scss_lint/auto_correct/correctors/hex_notation.rb

@@ -0,0 +1,16 @@
+module SCSSLint::AutoCorrect::Correctors
+  class HexNotation < Base
+    def call(contents)
+      contents.gsub(/#([A-Fa-f0-9]{6}|[A-Fa-f0-9]{3})/) { |m| correct_case(m) }
+    end
+
+    def correct_case(str)
+      return str.downcase if config['style'] == 'lowercase' || config['style'].nil?
+      str.upcase
+    end
+
+    def self.description
+      "Downcase hex color values"
+    end
+  end
+end

data/lib/scss_lint/auto_correct/correctors/indentation.rb

@@ -0,0 +1,35 @@
+module SCSSLint::AutoCorrect::Correctors
+  class Indentation < Base
+    def initialize(config = {})
+      super
+      @result = []
+      @level = 0
+      @indentation_symbol = "  "
+    end
+
+    def call(source)
+      source.lines.each { |line| visit_line(line) }
+      @result.join
+    end
+
+    def visit_line(line)
+      if line =~ /^\s*\/\//
+        @result << line
+        return
+      end
+      opening = line.include? "{"
+      closing = line.include? "}"
+      @level -= 1 if closing && !opening
+      @result << apply_indentation(line, @level)
+      @level += 1 if opening && !closing
+    end
+
+    def apply_indentation(line, level)
+      (@indentation_symbol * level) + line.lstrip
+    end
+
+    def self.description
+      "Fixes indentation"
+    end
+  end
+end

data/lib/scss_lint/auto_correct/correctors/leading_zero.rb

@@ -0,0 +1,37 @@
+module SCSSLint::AutoCorrect::Correctors
+  class LeadingZero < Base
+    def call(contents)
+      result = []
+      contents.lines.each do |line|
+        if line =~ /\s*\/\// # skip comments
+          result << line
+          next
+        end
+        mm = line.match(/(.+:)([\s\S]*)$/)
+        if mm
+          result << mm[1] + correct_values(mm[2])
+        else
+          result << line
+        end
+      end
+      result.join
+    end
+
+    def correct_values(str)
+      should_include = config.fetch('style', nil) == 'include_zero'
+      str.gsub(/-?(0?\.\d+)/) do |number|
+        if number[0] == "0" && !should_include
+          number[1..-1]
+        elsif number[0] != "0" && should_include
+          "0#{number}"
+        else
+          number
+        end
+      end
+    end
+
+    def self.description
+      "Checks for unnecessary leading zeros in numeric values with decimal points"
+    end
+  end
+end

data/lib/scss_lint/auto_correct/correctors/property_sort_order.rb

@@ -42,10 +42,6 @@ module SCSSLint::AutoCorrect::Correctors
       @sorter.sort_lines(lines)
     end
 
-    def self.linter_name
-      "PropertySortOrder"
-    end
-
     def self.description
       "Sorts properties correctly"
     end

data/lib/scss_lint/auto_correct/correctors/{newlines_for_each_selector.rb → single_line_per_selector.rb}

@@ -1,7 +1,7 @@
 module SCSSLint::AutoCorrect::Correctors
-  class NewlinesForEachSelector < Base
+  class SingleLinePerSelector < Base
     def call(contents)
-      contents.gsub(/([a-zA-Z0-9\[\]=~* ]+,){2,}.+{/) do |m|
+      contents.gsub(/([a-zA-Z0-9\[\]=~* ]+,){1,}.+{/) do |m|
         selectors = m.split(",")
         first = selectors.first.rstrip
         rest = selectors[1..-1].map { |s| " " * first.count(" ") + s.strip }

data/lib/scss_lint/auto_correct/correctors/{spacing_before_selectors.rb → space_before_brace.rb}

@@ -1,5 +1,5 @@
 module SCSSLint::AutoCorrect::Correctors
-  class SpacingBeforeSelectors < Base
+  class SpaceBeforeBrace < Base
     def call(contents)
       contents.gsub(/[a-zA-Z0-9%\[\]=*~]+ +{/) { |m| m.gsub(/ +/, ' ') }
     end

data/lib/scss_lint/auto_correct/correctors/string_quotes.rb

@@ -0,0 +1,16 @@
+module SCSSLint::AutoCorrect::Correctors
+  class StringQuotes < Base
+    def call(contents)
+      # contents.gsub("'", '"')
+      linter = SCSSLint::Linter::StringQuotes.new
+      contents.gsub(/"[^"]*"|'[^']*'/) do |mm|
+        unescaped = linter.__send__ :extract_string_without_quotes, mm
+        unescaped.dump
+      end
+    end
+
+    def self.description
+      "Use double quotes, rather than single quotes"
+    end
+  end
+end

data/lib/scss_lint/auto_correct/correctors/{lines_end_with_semicolons.rb → trailing_semicolon.rb}

@@ -1,9 +1,12 @@
 module SCSSLint::AutoCorrect::Correctors
-  class LinesEndWithSemicolons < Base
+  class TrailingSemicolon < Base
     def call(contents)
       # if a line ends with a letter, it means it should end with a semicolon
       # (and it contains a : but no comments (// and *) and some buggy expressions (&))
-      contents.gsub(/^[^\/*&]+:[^\/*&]+[A-Za-z0-9-]$/, '\0;')
+      contents.
+        gsub(/^[^\/*&]+:[^\/*&]+[A-Za-z0-9-]$/, '\0;').
+        gsub(/ +;/, ';').
+        gsub(/;+/, ';')
     end
 
     def self.description

data/lib/scss_lint/auto_correct/correctors/{variable_names.rb → variable_name.rb}

@@ -1,9 +1,13 @@
 module SCSSLint::AutoCorrect::Correctors
-  class VariableNames < Base
+  class VariableName < Base
     def call(contents)
       contents.gsub(/\$[a-z_-]+/) { |m| m.gsub('_', '-') }
     end
 
+    def self.linter_name
+      "NameFormat"
+    end
+
     def self.description
       "Use dashes instead of underscores in variable names"
     end

data/lib/scss_lint/auto_correct/version.rb

@@ -1,4 +1,4 @@
 # Defines the gem version.
 module SCSSLint::AutoCorrect
-  VERSION = '1.1.0'.freeze
+  VERSION = '1.2.0'.freeze
 end

data/test/fixtures/double_semicolon.scss

@@ -0,0 +1,3 @@
+.lorem {
+  height: 1px;;
+}

data/test/fixtures/double_semicolon_corrected.scss

@@ -0,0 +1,3 @@
+.lorem {
+  height: 1px;
+}

data/test/fixtures/empty_line_between_blocks.scss

@@ -0,0 +1,14 @@
+.lorem {
+  height: 1px;
+}
+.ipsum {
+  height: 1px;
+}
+.pig {
+  .peppa {
+    height: 1px;
+  }
+  .george {
+    height: 1px;
+  }
+}

data/test/fixtures/empty_line_between_blocks_corrected.scss

@@ -0,0 +1,17 @@
+.lorem {
+  height: 1px;
+}
+
+.ipsum {
+  height: 1px;
+}
+
+.pig {
+  .peppa {
+    height: 1px;
+  }
+
+  .george {
+    height: 1px;
+  }
+}

data/test/fixtures/has_leading_zero.scss

@@ -0,0 +1,3 @@
+.lorem {
+  height: 0.24em;
+}

data/test/fixtures/has_leading_zero_corrected.scss

@@ -0,0 +1,3 @@
+.lorem {
+  height: .24em;
+}

data/test/fixtures/has_leading_zero_with_comments.scss

@@ -0,0 +1,5 @@
+# LICENSE-2.0
+# This is v0.0.7
+.lorem {
+  height: 1px;
+}

data/test/fixtures/has_underscores_for_variable_names.scss

@@ -0,0 +1,5 @@
+$my_var: 42;
+
+.lorem {
+  height: $my_var;
+}

data/test/fixtures/has_underscores_for_variable_names_corrected.scss

@@ -0,0 +1,5 @@
+$my-var: 42;
+
+.lorem {
+  height: $my-var;
+}

data/test/fixtures/indentation.scss

@@ -0,0 +1,6 @@
+.lorem {
+.ipsum {
+color: #fff;
+&:hover { color: #f00; }
+}
+}

data/test/fixtures/indentation_corrected.scss

@@ -0,0 +1,6 @@
+.lorem {
+  .ipsum {
+    color: #fff;
+    &:hover { color: #f00; }
+  }
+}

data/test/fixtures/indentation_with_comments.scss

@@ -0,0 +1,6 @@
+// .lorem {
+// .ipsum {
+// color: #fff;
+// &:hover { color: #f00; }
+// }
+// }

data/test/fixtures/missing_semicolon.scss

@@ -0,0 +1,3 @@
+.lorem {
+  height: 1px
+}

data/test/fixtures/missing_semicolon_corrected.scss

@@ -0,0 +1,3 @@
+.lorem {
+  height: 1px;
+}

data/test/fixtures/multiple_selectors_on_same_line.scss

@@ -0,0 +1,3 @@
+.lorem, .ipsum {
+  height: 1px;
+}

data/test/fixtures/multiple_selectors_on_same_line_corrected.scss

@@ -0,0 +1,4 @@
+.lorem,
+.ipsum {
+  height: 1px;
+}

data/test/fixtures/space_before_opening_brace.scss

@@ -0,0 +1,3 @@
+.lorem    {
+  height: 1px;
+}

data/test/fixtures/space_before_opening_brace_corrected.scss

@@ -0,0 +1,3 @@
+.lorem {
+  height: 1px;
+}

data/test/fixtures/space_before_semicolon.scss

@@ -0,0 +1,3 @@
+.lorem {
+  height: 1px ;
+}

data/test/fixtures/space_before_semicolon_corrected.scss

@@ -0,0 +1,3 @@
+.lorem {
+  height: 1px;
+}

data/test/unit/cli_test.rb

@@ -23,14 +23,14 @@ class CliTest < Minitest::Test
 
     it "allows running a specific correctors only" do
       instance = cli.new
-      instance.parse_args! ["--downcase-colors", "--format-newlines"]
+      instance.parse_args! ["--hex-notation", "--empty-line-between-blocks"]
       assert_equal 2, instance.selected_correctors.count
     end
 
     it "allows skipping a specific corrector" do
       instance = cli.new
       all = instance.selected_correctors.count
-      instance.parse_args! ["--no-downcase-colors"]
+      instance.parse_args! ["--no-hex-notation"]
       assert_equal all - 1, instance.selected_correctors.count
     end
 

data/test/unit/color_keyword.rb

@@ -0,0 +1,16 @@
+require_relative "../test_helper"
+
+class ColorKeywordTest < Minitest::Test
+  describe "SCSSLint::AutoCorrect::Correctors::ColorKeyword" do
+    let(:corrector) { SCSSLint::AutoCorrect::Correctors::ColorKeyword }
+
+    it "basically works" do
+      corrector.new
+    end
+
+    it "transforms name to hex" do
+      output = corrector.new.call "white"
+      assert_equal "#fff", output
+    end
+  end
+end

data/test/unit/empty_line_between_blocks_test.rb

@@ -0,0 +1,18 @@
+require_relative "../test_helper"
+
+class EmptyLineBetweenBlocksTest < Minitest::Test
+  describe "SCSSLint::AutoCorrect::Correctors::EmptyLineBetweenBlocks" do
+    let(:corrector) { SCSSLint::AutoCorrect::Correctors::EmptyLineBetweenBlocks }
+
+    it "basically works" do
+      corrector.new
+    end
+
+    it "process input" do
+      input = load_file "empty_line_between_blocks.scss"
+      expected = load_file "empty_line_between_blocks_corrected.scss"
+      output = corrector.new.call input
+      assert_equal expected, output
+    end
+  end
+end

data/test/unit/hex_length_test.rb

@@ -0,0 +1,26 @@
+require_relative "../test_helper"
+
+class HexLengthTest < Minitest::Test
+  describe "SCSSLint::AutoCorrect::Correctors::HexLength" do
+    let(:corrector) { SCSSLint::AutoCorrect::Correctors::HexLength }
+
+    it "basically works" do
+      corrector.new
+    end
+
+    it "shortens long hex" do
+      output = corrector.new.call "#ffffff"
+      assert_equal "#fff", output
+    end
+
+    it "ignores unshortable hex" do
+      output = corrector.new.call "#fedeabe"
+      assert_equal "#fedeabe", output
+    end
+
+    it "ignores partially unshortable hex" do
+      output = corrector.new.call "#ffdeabe"
+      assert_equal "#ffdeabe", output
+    end
+  end
+end

data/test/unit/{downcase_colors_test.rb → hex_notation_test.rb}

@@ -1,8 +1,8 @@
 require_relative "../test_helper"
 
-class DowncaseColorsTest < Minitest::Test
-  describe "SCSSLint::AutoCorrect::Correctors::DowncaseColors" do
-    let(:corrector) { SCSSLint::AutoCorrect::Correctors::DowncaseColors }
+class HexNotationTest < Minitest::Test
+  describe "SCSSLint::AutoCorrect::Correctors::HexNotation" do
+    let(:corrector) { SCSSLint::AutoCorrect::Correctors::HexNotation }
 
     it "basically works" do
       corrector.new

data/test/unit/indentation_test.rb

@@ -0,0 +1,24 @@
+require_relative "../test_helper"
+
+class IndentationTest < Minitest::Test
+  describe "SCSSLint::AutoCorrect::Correctors::Indentation" do
+    let(:corrector) { SCSSLint::AutoCorrect::Correctors::Indentation }
+
+    it "basically works" do
+      corrector.new
+    end
+
+    it "process input" do
+      input = load_file "indentation.scss"
+      expected = load_file "indentation_corrected.scss"
+      output = corrector.new.call input
+      assert_equal expected, output
+    end
+
+    it "ignores commented lines" do
+      input = load_file "indentation_with_comments.scss"
+      output = corrector.new.call input
+      assert_equal input, output
+    end
+  end
+end

data/test/unit/leading_zero_test.rb

@@ -0,0 +1,34 @@
+require_relative "../test_helper"
+
+class LeadingZeroTest < Minitest::Test
+  describe "SCSSLint::AutoCorrect::Correctors::LeadingZero" do
+    let(:corrector) { SCSSLint::AutoCorrect::Correctors::LeadingZero }
+
+    it "basically works" do
+      corrector.new
+    end
+
+    it "removes leading zeros" do
+      input = load_file "has_leading_zero.scss"
+      expected = load_file "has_leading_zero_corrected.scss"
+      output = corrector.new.call input
+      assert_equal expected, output
+    end
+
+    it "appends leading zeros, if configured" do
+      config = {
+        'style' => 'include_zero'
+      }
+      input = load_file "has_leading_zero_corrected.scss"
+      expected = load_file "has_leading_zero.scss"
+      output = corrector.new(config).call input
+      assert_equal expected, output
+    end
+
+    it "skips over tasty comments" do
+      input = load_file "has_leading_zero_with_comments.scss"
+      output = corrector.new.call input
+      assert_equal input, output
+    end
+  end
+end

data/test/unit/single_line_per_selector_test.rb

@@ -0,0 +1,18 @@
+require_relative "../test_helper"
+
+class SingleLinePerSelectorTest < Minitest::Test
+  describe "SCSSLint::AutoCorrect::Correctors::SingleLinePerSelector" do
+    let(:corrector) { SCSSLint::AutoCorrect::Correctors::SingleLinePerSelector }
+
+    it "basically works" do
+      corrector.new
+    end
+
+    it "puts each selector on its own line" do
+      input = load_file "multiple_selectors_on_same_line.scss"
+      expected = load_file "multiple_selectors_on_same_line_corrected.scss"
+      output = corrector.new.call input
+      assert_equal expected, output
+    end
+  end
+end

data/test/unit/space_before_brace_test.rb

@@ -0,0 +1,18 @@
+require_relative "../test_helper"
+
+class SpaceBeforeBraceTest < Minitest::Test
+  describe "SCSSLint::AutoCorrect::Correctors::SpaceBeforeBrace" do
+    let(:corrector) { SCSSLint::AutoCorrect::Correctors::SpaceBeforeBrace }
+
+    it "basically works" do
+      corrector.new
+    end
+
+    it "puts each selector on its own line" do
+      input = load_file "space_before_opening_brace.scss"
+      expected = load_file "space_before_opening_brace_corrected.scss"
+      output = corrector.new.call input
+      assert_equal expected, output
+    end
+  end
+end

data/test/unit/string_quotes_test.rb

@@ -0,0 +1,24 @@
+require_relative "../test_helper"
+
+class StringQuotesTest < Minitest::Test
+  describe "SCSSLint::AutoCorrect::Correctors::StringQuotes" do
+    let(:corrector) { SCSSLint::AutoCorrect::Correctors::StringQuotes }
+
+    it "basically works" do
+      corrector.new
+    end
+
+    it "converts sigle quotes to doubles" do
+      input = ".lorem { color: 'foo'; }"
+      expected = '.lorem { color: "foo"; }'
+      output = corrector.new.call(input)
+      assert_equal expected, output
+    end
+
+    it "respectfully deals with nested quotes" do
+      input = ".lorem { color: \"foo 'bar'\"; }"
+      output = corrector.new.call(input)
+      assert_equal input, output
+    end
+  end
+end

data/test/unit/trailing_semicolon_test.rb

@@ -0,0 +1,32 @@
+require_relative "../test_helper"
+
+class TrailingSemicolonTest < Minitest::Test
+  describe "SCSSLint::AutoCorrect::Correctors::TrailingSemicolon" do
+    let(:corrector) { SCSSLint::AutoCorrect::Correctors::TrailingSemicolon }
+
+    it "basically works" do
+      corrector.new
+    end
+
+    it "adds missing semicolons" do
+      input = load_file "missing_semicolon.scss"
+      expected = load_file "missing_semicolon_corrected.scss"
+      output = corrector.new.call input
+      assert_equal expected, output
+    end
+
+    it "removes extra space before semicolon" do
+      input = load_file "space_before_semicolon.scss"
+      expected = load_file "space_before_semicolon_corrected.scss"
+      output = corrector.new.call input
+      assert_equal expected, output
+    end
+
+    it "removes double semicolon" do
+      input = load_file "double_semicolon.scss"
+      expected = load_file "double_semicolon_corrected.scss"
+      output = corrector.new.call input
+      assert_equal expected, output
+    end
+  end
+end

data/test/unit/variable_name_test.rb

@@ -0,0 +1,18 @@
+require_relative "../test_helper"
+
+class VariableNameTest < Minitest::Test
+  describe "SCSSLint::AutoCorrect::Correctors::VariableName" do
+    let(:corrector) { SCSSLint::AutoCorrect::Correctors::VariableName }
+
+    it "basically works" do
+      corrector.new
+    end
+
+    it "does its duty" do
+      input = load_file "has_underscores_for_variable_names.scss"
+      expected = load_file "has_underscores_for_variable_names_corrected.scss"
+      output = corrector.new.call input
+      assert_equal expected, output
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: scss_lint-auto_correct
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Dorian Marié
@@ -40,28 +40,58 @@ files:
 - lib/scss_lint/auto_correct/cli.rb
 - lib/scss_lint/auto_correct/correctors.rb
 - lib/scss_lint/auto_correct/correctors/base.rb
-- lib/scss_lint/auto_correct/correctors/double_quoting.rb
-- lib/scss_lint/auto_correct/correctors/downcase_colors.rb
-- lib/scss_lint/auto_correct/correctors/format_newlines.rb
-- lib/scss_lint/auto_correct/correctors/lines_end_with_semicolons.rb
-- lib/scss_lint/auto_correct/correctors/newlines_for_each_selector.rb
-- lib/scss_lint/auto_correct/correctors/no_spaces_before_semicolons.rb
+- lib/scss_lint/auto_correct/correctors/color_keyword.rb
+- lib/scss_lint/auto_correct/correctors/empty_line_between_blocks.rb
+- lib/scss_lint/auto_correct/correctors/hex_length.rb
+- lib/scss_lint/auto_correct/correctors/hex_notation.rb
+- lib/scss_lint/auto_correct/correctors/indentation.rb
+- lib/scss_lint/auto_correct/correctors/leading_zero.rb
 - lib/scss_lint/auto_correct/correctors/property_sort_order.rb
-- lib/scss_lint/auto_correct/correctors/short_versions_of_numbers.rb
-- lib/scss_lint/auto_correct/correctors/shorter_versions_of_colors.rb
-- lib/scss_lint/auto_correct/correctors/spacing_before_selectors.rb
-- lib/scss_lint/auto_correct/correctors/use_hex_instead_of_color_keywords.rb
-- lib/scss_lint/auto_correct/correctors/variable_names.rb
+- lib/scss_lint/auto_correct/correctors/single_line_per_selector.rb
+- lib/scss_lint/auto_correct/correctors/space_before_brace.rb
+- lib/scss_lint/auto_correct/correctors/string_quotes.rb
+- lib/scss_lint/auto_correct/correctors/trailing_semicolon.rb
+- lib/scss_lint/auto_correct/correctors/variable_name.rb
 - lib/scss_lint/auto_correct/property_sorter.rb
 - lib/scss_lint/auto_correct/version.rb
+- test/fixtures/double_semicolon.scss
+- test/fixtures/double_semicolon_corrected.scss
+- test/fixtures/empty_line_between_blocks.scss
+- test/fixtures/empty_line_between_blocks_corrected.scss
+- test/fixtures/has_leading_zero.scss
+- test/fixtures/has_leading_zero_corrected.scss
+- test/fixtures/has_leading_zero_with_comments.scss
+- test/fixtures/has_underscores_for_variable_names.scss
+- test/fixtures/has_underscores_for_variable_names_corrected.scss
 - test/fixtures/has_uppercase_colors.scss
+- test/fixtures/indentation.scss
+- test/fixtures/indentation_corrected.scss
+- test/fixtures/indentation_with_comments.scss
+- test/fixtures/missing_semicolon.scss
+- test/fixtures/missing_semicolon_corrected.scss
+- test/fixtures/multiple_selectors_on_same_line.scss
+- test/fixtures/multiple_selectors_on_same_line_corrected.scss
+- test/fixtures/space_before_opening_brace.scss
+- test/fixtures/space_before_opening_brace_corrected.scss
+- test/fixtures/space_before_semicolon.scss
+- test/fixtures/space_before_semicolon_corrected.scss
 - test/fixtures/unsorted_properties.scss
 - test/fixtures/unsorted_properties_corrected.scss
 - test/fixtures/unsorted_properties_disable_rule.scss
 - test/test_helper.rb
 - test/unit/cli_test.rb
-- test/unit/downcase_colors_test.rb
+- test/unit/color_keyword.rb
+- test/unit/empty_line_between_blocks_test.rb
+- test/unit/hex_length_test.rb
+- test/unit/hex_notation_test.rb
+- test/unit/indentation_test.rb
+- test/unit/leading_zero_test.rb
 - test/unit/property_sort_order_test.rb
+- test/unit/single_line_per_selector_test.rb
+- test/unit/space_before_brace_test.rb
+- test/unit/string_quotes_test.rb
+- test/unit/trailing_semicolon_test.rb
+- test/unit/variable_name_test.rb
 homepage: https://github.com/Dorian/scss-lint-auto-correct
 licenses:
 - MIT
@@ -87,11 +117,41 @@ signing_key:
 specification_version: 4
 summary: Auto corrections for SCSS lint tool
 test_files:
+- test/fixtures/double_semicolon.scss
+- test/fixtures/double_semicolon_corrected.scss
+- test/fixtures/empty_line_between_blocks.scss
+- test/fixtures/empty_line_between_blocks_corrected.scss
+- test/fixtures/has_leading_zero.scss
+- test/fixtures/has_leading_zero_corrected.scss
+- test/fixtures/has_leading_zero_with_comments.scss
+- test/fixtures/has_underscores_for_variable_names.scss
+- test/fixtures/has_underscores_for_variable_names_corrected.scss
 - test/fixtures/has_uppercase_colors.scss
+- test/fixtures/indentation.scss
+- test/fixtures/indentation_corrected.scss
+- test/fixtures/indentation_with_comments.scss
+- test/fixtures/missing_semicolon.scss
+- test/fixtures/missing_semicolon_corrected.scss
+- test/fixtures/multiple_selectors_on_same_line.scss
+- test/fixtures/multiple_selectors_on_same_line_corrected.scss
+- test/fixtures/space_before_opening_brace.scss
+- test/fixtures/space_before_opening_brace_corrected.scss
+- test/fixtures/space_before_semicolon.scss
+- test/fixtures/space_before_semicolon_corrected.scss
 - test/fixtures/unsorted_properties.scss
 - test/fixtures/unsorted_properties_corrected.scss
 - test/fixtures/unsorted_properties_disable_rule.scss
 - test/test_helper.rb
 - test/unit/cli_test.rb
-- test/unit/downcase_colors_test.rb
+- test/unit/color_keyword.rb
+- test/unit/empty_line_between_blocks_test.rb
+- test/unit/hex_length_test.rb
+- test/unit/hex_notation_test.rb
+- test/unit/indentation_test.rb
+- test/unit/leading_zero_test.rb
 - test/unit/property_sort_order_test.rb
+- test/unit/single_line_per_selector_test.rb
+- test/unit/space_before_brace_test.rb
+- test/unit/string_quotes_test.rb
+- test/unit/trailing_semicolon_test.rb
+- test/unit/variable_name_test.rb

data/lib/scss_lint/auto_correct/correctors/double_quoting.rb

@@ -1,11 +0,0 @@
-module SCSSLint::AutoCorrect::Correctors
-  class DoubleQuoting < Base
-    def call(contents)
-      contents.gsub("'", '"')
-    end
-
-    def self.description
-      "Use double quotes, rather than single quotes"
-    end
-  end
-end

data/lib/scss_lint/auto_correct/correctors/downcase_colors.rb

@@ -1,11 +0,0 @@
-module SCSSLint::AutoCorrect::Correctors
-  class DowncaseColors < Base
-    def call(contents)
-      contents.gsub(/#([A-Fa-f0-9]{6}|[A-Fa-f0-9]{3})/) { |m| m.downcase }
-    end
-
-    def self.description
-      "Downcase hex color values"
-    end
-  end
-end

data/lib/scss_lint/auto_correct/correctors/no_spaces_before_semicolons.rb

@@ -1,11 +0,0 @@
-module SCSSLint::AutoCorrect::Correctors
-  class NoSpacesBeforeSemicolons < Base
-    def call(contents)
-      contents.gsub(/ +;/, ';')
-    end
-
-    def self.description
-      "Remove spaces before semicolons"
-    end
-  end
-end

data/lib/scss_lint/auto_correct/correctors/short_versions_of_numbers.rb

@@ -1,13 +0,0 @@
-module SCSSLint::AutoCorrect::Correctors
-  class ShortVersionsOfNumbers < Base
-    def call(contents)
-      # do not match 1.04 or v1.0.0 or LICENSE-2.0
-      # (this is pure dark magic)
-      contents.gsub(/([^-v[0-9]?\.?)])([0-9])\.0+([^0-9])/, '\1\2\3')
-    end
-
-    def self.description
-      ""
-    end
-  end
-end

data/lib/scss_lint/auto_correct/correctors/use_hex_instead_of_color_keywords.rb

@@ -1,30 +0,0 @@
-module SCSSLint::AutoCorrect::Correctors
-  class UseHexInsteadOfColorKeywords < Base
-    def call(contents)
-      colors = self.class.data
-      regexp = Regexp.new("^([^\/]+ )(#{colors.keys.join('|')})([\) ;].*)$")
-
-      contents.gsub(regexp) do
-        Regexp.last_match[1] + colors[Regexp.last_match[2]] + Regexp.last_match[3]
-      end
-    end
-
-    def self.data
-      return @data if @data
-      colors_file = File.expand_path(File.join(File.dirname(__FILE__), "color-keywords.csv"))
-      @data = Hash[File.read(colors_file).lines.map { |l| l.gsub("\n", '').split(',') }]
-    end
-
-    def self.description
-      "Replace named colors with their hex values"
-    end
-
-    def self.priority
-      5
-    end
-
-    def self.linter_name
-      "ColorKeyword"
-    end
-  end
-end