overcommit 0.20.0 → 0.21.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: dcf7ae4335496b2ce26f507167d772d304a7afd2
-  data.tar.gz: 25ef2ce0e6350399b0814b94b7ac78929923b2bc
+  metadata.gz: ea70d2648519c5c2f90bfa96c690a45414c1df2a
+  data.tar.gz: 65e1c6ccda24f2eb76fa6982439112964414dd4e
 SHA512:
-  metadata.gz: ab8e8e06501b756741038cc6ea651d215ffd47201390dbff2351fdc0fc668d51378402cf9f08d13ba887c50fa74cfbca7cafa954d908f5ca9f6f1405adc25eed
-  data.tar.gz: 9043450a453504dc2a41110bd184ef5d08ec4580cac9f78f29bfa5eb8cea715f84b660e1e9e024b7cad42049d564e771f2a475c93b0f33bb5c6882b2f6a9fb9c
+  metadata.gz: bae413ee48cbf60e6de660988fbdc8dc2c3a8044249d7089618500a033629df0b0fb6d22a8465732d774c043c2a8fd0abba3d0f30c618041f99b8292885303af
+  data.tar.gz: 26f3e6ffb0999a3f25573a53b48b8e2f236ba880423598845c43e0609e0c8aeec9c9236d7caee8f659e01b886f38461931e99781d20920e0690119cbddfc9ae4

data/config/default.yml

@@ -113,6 +113,7 @@ PreCommit:
 
   HardTabs:
     description: 'Checking for hard tabs'
+    quiet: true
     exclude:
       - '**/*.go'
       - '**/Makefile'
@@ -160,9 +161,11 @@ PreCommit:
 
   MergeConflicts:
     description: 'Checking for merge conflicts'
+    quiet: true
 
   PryBinding:
     description: 'Checking for instances of binding.pry'
+    quiet: true
     include:
       - '**/*.rb'
       - '**/*.rake'
@@ -208,6 +211,11 @@ PreCommit:
     install_command: 'gem install scss-lint'
     include: '**/*.scss'
 
+  ShellCheck:
+    description: 'Analyzing with ShellCheck'
+    required_executable: 'shellcheck'
+    include: '**/*.sh'
+
   TrailingWhitespace:
     description: 'Checking for trailing whitespace'
     exclude:
@@ -215,8 +223,8 @@ PreCommit:
 
   TravisLint:
     description: 'Checking Travis CI configuration'
-    required_executable: 'travis-lint'
-    install_command: 'gem install travis-lint'
+    required_executable: 'travis'
+    install_command: 'gem install travis'
     include: '.travis.yml'
 
   YamlSyntax:

data/lib/overcommit/hook/pre_commit/jscs.rb

@@ -3,7 +3,7 @@ module Overcommit::Hook::PreCommit
   # files.
   class Jscs < Base
     def run
-      result = execute(%W[#{executable} --reporter=inline] + applicable_files)
+      result = execute(%W[#{executable} --reporter=inline --verbose] + applicable_files)
       return :pass if result.success?
 
       if result.status == 1
@@ -11,6 +11,8 @@ module Overcommit::Hook::PreCommit
         return :warn, result.stderr.chomp
       end
 
+      # example message:
+      #   path/to/file.js: line 7, col 0, ruleName: Error message
       extract_messages(
         result.stdout.split("\n"),
         /^(?<file>[^:]+):[^\d]+(?<line>\d+)/,

data/lib/overcommit/hook/pre_commit/shell_check.rb

@@ -0,0 +1,19 @@
+module Overcommit::Hook::PreCommit
+  # Runs `shellcheck` against any modified shell script files.
+  class ShellCheck < Base
+    MESSAGE_TYPE_CATEGORIZER = lambda do |type|
+      type.include?('note') ? :warning : :error
+    end
+
+    def run
+      result = execute(%W[#{executable} --format=gcc] + applicable_files)
+      return :pass if result.success?
+
+      extract_messages(
+        result.stdout.split("\n"),
+        /^(?<file>[^:]+):(?<line>\d+):[^ ]+ (?<type>[^ ]+)/,
+        MESSAGE_TYPE_CATEGORIZER,
+      )
+    end
+  end
+end

data/lib/overcommit/hook/pre_commit/travis_lint.rb

@@ -2,10 +2,10 @@ module Overcommit::Hook::PreCommit
   # Runs `travis-lint` against any modified Travis CI files.
   class TravisLint < Base
     def run
-      result = execute([executable] + applicable_files)
+      result = execute([executable, 'lint'] + applicable_files)
       return :pass if result.success?
 
-      [:fail, result.stdout.strip]
+      [:fail, (result.stdout + result.stderr).strip]
     end
   end
 end

data/lib/overcommit/version.rb

@@ -1,4 +1,4 @@
 # Defines the gem version.
 module Overcommit
-  VERSION = '0.20.0'
+  VERSION = '0.21.0'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: overcommit
 version: !ruby/object:Gem::Version
-  version: 0.20.0
+  version: 0.21.0
 platform: ruby
 authors:
 - Causes Engineering
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-17 00:00:00.000000000 Z
+date: 2015-01-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: childprocess
@@ -53,6 +53,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: travis
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.7'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.7'
 description: Utility to install, configure, and extend Git hooks
 email:
 - eng@causes.com
@@ -110,6 +124,7 @@ files:
 - lib/overcommit/hook/pre_commit/reek.rb
 - lib/overcommit/hook/pre_commit/rubocop.rb
 - lib/overcommit/hook/pre_commit/scss_lint.rb
+- lib/overcommit/hook/pre_commit/shell_check.rb
 - lib/overcommit/hook/pre_commit/trailing_whitespace.rb
 - lib/overcommit/hook/pre_commit/travis_lint.rb
 - lib/overcommit/hook/pre_commit/yaml_syntax.rb