CODE REVIEW

Author: Ruffeng

README.md

@@ -546,7 +546,7 @@ issue](https://github.com/brigade/overcommit/issues/238) for more details.
 * [SlimLint](lib/overcommit/hook/pre_commit/slim_lint.rb)
 * [Sqlint](lib/overcommit/hook/pre_commit/sqlint.rb)
 * [Standard](lib/overcommit/hook/pre_commit/standard.rb)
-* [StyleLint](lib/overcommit/hook/pre_commit/style_lint.rb)
+* [StyleLint](lib/overcommit/hook/pre_commit/stylelint.rb)
 * [TrailingWhitespace](lib/overcommit/hook/pre_commit/trailing_whitespace.rb)
 * [TravisLint](lib/overcommit/hook/pre_commit/travis_lint.rb)
 * [TsLint](lib/overcommit/hook/pre_commit/ts_lint.rb)

config/default.yml

@@ -712,10 +712,10 @@ PreCommit:
     include: '**/*.js'
 
   StyleLint:
-    enabled: true
+    enabled: false
     description: 'Check styles with Stylelint'
     require_executable: 'stylelint'
-    flags: ['-f compact']
+    flags: ['-f', 'compact']
     install_command: 'npm install -g stylelint'
     include:
       - '**/*.scss'

lib/overcommit/hook/pre_commit/style_lint.rb lib/overcommit/hook/pre_commit/stylelint.rb

@@ -2,7 +2,7 @@ module Overcommit::Hook::PreCommit
   # Runs `stylelint` against any modified CSS file.
   #
   # @see https://github.com/stylelint/stylelint
-  class StyleLint < Base
+  class Stylelint < Base
     # example of output:
     # index.css: line 4, col 4, error - Expected indentation of 2 spaces (indentation)
 

spec/overcommit/hook/pre_commit/style_lint_spec.rb spec/overcommit/hook/pre_commit/stylelint_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe Overcommit::Hook::PreCommit::StyleLint do
+describe Overcommit::Hook::PreCommit::Stylelint do
   let(:config)  { Overcommit::ConfigurationLoader.default_configuration }
   let(:context) { double('context') }
   subject { described_class.new(config, context) }