overcommit 0.27.0 → 0.28.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b15d613ceeafdf7cf3c762c611b309b596bf9fd9
-  data.tar.gz: 89d01be1ecf271fc7f8f6b6472fb6e056e8980c7
+  metadata.gz: 74c5fbc6dd718ba856a0c0ad6255f9e6e0d6f72c
+  data.tar.gz: 9b40e46e88199c43d8c708202b872ea5ce1b52fc
 SHA512:
-  metadata.gz: 5a665c68e8e3066f1f4f2b62997474b20f910360a7aadf2b24d5c2caad64260a3c2f485ffbd9174b7681fbd24a743a07a97820a2c4fed38e5d54c00c95ceccd8
-  data.tar.gz: a570dcdcf2ac15e352f837ebb404819f197d74428881586f2c2eec1f6a4f4eebfbf5e4c80536cb38fe2a6ca941b5de129357ae0437c8b739a747738a4f2d1baf
+  metadata.gz: 278cfa2c64204a0bfe3b9c5e0dfae81396ec1317fadaa1fb4364bf979795c642e47c39c16b2ba64dc2602f1377aeb3073011c67ab091540cc7027e931f3500bc
+  data.tar.gz: b9908d3670f23dcc0c4766da676e44642eb5f211147de22801690c7ad679ba589d9f8b5f14210dc2a6ad65a06fc6ecb82807e2517efafbacf5443a550963d91b

data/bin/overcommit

@@ -1,6 +1,5 @@
 #!/usr/bin/env ruby
 
-require 'overcommit'
 require 'overcommit/cli'
 
 logger = Overcommit::Logger.new(STDOUT)

data/config/default.yml

@@ -176,8 +176,7 @@ PreCommit:
     enabled: false
     description: 'Analyzing with coffeelint'
     required_executable: 'coffeelint'
-    required_library: 'json'
-    flags: ['--reporter=raw']
+    flags: ['--reporter=csv']
     install_command: 'npm install -g coffeelint'
     include: '**/*.coffee'
 
@@ -197,6 +196,11 @@ PreCommit:
     install_command: 'npm install -g eslint'
     include: '**/*.js'
 
+  ExecutePermissions:
+    enabled: false
+    description: 'Checking for file execute permissions'
+    quiet: true
+
   GoLint:
     enabled: false
     description: 'Analyzing with golint'
@@ -226,6 +230,13 @@ PreCommit:
     required_executable: 'grep'
     flags: ['-IHn', "\t"]
 
+  Hlint:
+    enabled: false
+    description: 'Analyzing with hlint'
+    required_executable: 'hlint'
+    install_command: 'cabal install hlint'
+    include: '**/*.hs'
+
   HtmlHint:
     enabled: false
     description: 'Analyzing with HTMLHint'
@@ -325,6 +336,17 @@ PreCommit:
     install_command: 'pip install pep8'
     include: '**/*.py'
 
+  PuppetLint:
+    enabled: false
+    description: 'Analyzing with puppet-lint'
+    required_executable: 'puppet-lint'
+    install_command: 'gem install puppet-lint'
+    flags:
+      - '--log-format="%{fullpath}:%{line}:%{column}:%{KIND}: %{message} (%{check})"'
+      - '--fail-on-warnings'
+      - '--error-level=all'
+    include: '**/*.pp'
+
   Pyflakes:
     enabled: false
     description: 'Analyzing with pyflakes'
@@ -436,6 +458,13 @@ PreCommit:
     install_command: 'gem install slim_lint'
     include: '**/*.slim'
 
+  Sqlint:
+    enabled: false
+    description: 'Analyzing with sqlint'
+    required_executable: 'sqlint'
+    install_command: 'gem install sqlint'
+    include: '**/*.sql'
+
   Standard:
     enabled: false
     description: 'Analyzing with standard'
@@ -521,12 +550,43 @@ PostCheckout:
     required: false
     quiet: false
 
+  BowerInstall:
+    enabled: false
+    description: 'Installing bower dependencies'
+    requires_files: true
+    required_executable: 'bower'
+    install_command: 'npm install -g bower'
+    flags: ['install']
+    include: 'bower.json'
+
+  BundleInstall:
+    enabled: false
+    description: 'Installing Bundler dependencies'
+    requires_files: true
+    required_executable: 'bundle'
+    install_command: 'gem install bundler'
+    flags: ['install']
+    include:
+      - 'Gemfile'
+      - 'Gemfile.lock'
+      - '*.gemspec'
+
   IndexTags:
     enabled: false
     description: 'Generating tags file from source'
     quiet: true
     required_executable: 'ctags'
 
+  NpmInstall:
+    enabled: false
+    description: 'Installing NPM dependencies'
+    requires_files: true
+    required_executable: 'npm'
+    flags: ['install']
+    include:
+      - 'package.json'
+      - 'npm-shrinkwrap.json'
+
   SubmoduleStatus:
     enabled: false
     description: 'Checking submodule status'
@@ -540,6 +600,27 @@ PostCommit:
     required: false
     quiet: false
 
+  BowerInstall:
+    enabled: false
+    description: 'Installing bower dependencies'
+    requires_files: true
+    required_executable: 'bower'
+    install_command: 'npm install -g bower'
+    flags: ['install']
+    include: 'bower.json'
+
+  BundleInstall:
+    enabled: false
+    description: 'Installing Bundler dependencies'
+    requires_files: true
+    required_executable: 'bundle'
+    install_command: 'gem install bundler'
+    flags: ['install']
+    include:
+      - 'Gemfile'
+      - 'Gemfile.lock'
+      - '*.gemspec'
+
   GitGuilt:
     enabled: false
     description: 'Calculating changes in blame since last commit'
@@ -554,6 +635,16 @@ PostCommit:
     quiet: true
     required_executable: 'ctags'
 
+  NpmInstall:
+    enabled: false
+    description: 'Installing NPM dependencies'
+    requires_files: true
+    required_executable: 'npm'
+    flags: ['install']
+    include:
+      - 'package.json'
+      - 'npm-shrinkwrap.json'
+
   SubmoduleStatus:
     enabled: false
     description: 'Checking submodule status'
@@ -566,12 +657,43 @@ PostMerge:
     requires_files: false
     quiet: false
 
+  BowerInstall:
+    enabled: false
+    description: 'Installing bower dependencies'
+    requires_files: true
+    required_executable: 'bower'
+    install_command: 'npm install -g bower'
+    flags: ['install']
+    include: 'bower.json'
+
+  BundleInstall:
+    enabled: false
+    description: 'Installing Bundler dependencies'
+    requires_files: true
+    required_executable: 'bundle'
+    install_command: 'gem install bundler'
+    flags: ['install']
+    include:
+      - 'Gemfile'
+      - 'Gemfile.lock'
+      - '*.gemspec'
+
   IndexTags:
     enabled: false
     description: 'Generating tags file from source'
     quiet: true
     required_executable: 'ctags'
 
+  NpmInstall:
+    enabled: false
+    description: 'Installing NPM dependencies'
+    requires_files: true
+    required_executable: 'npm'
+    flags: ['install']
+    include:
+      - 'package.json'
+      - 'npm-shrinkwrap.json'
+
   SubmoduleStatus:
     enabled: false
     description: 'Checking submodule status'
@@ -584,12 +706,43 @@ PostRewrite:
     requires_files: false
     quiet: false
 
+  BowerInstall:
+    enabled: false
+    description: 'Installing bower dependencies'
+    requires_files: true
+    required_executable: 'bower'
+    install_command: 'npm install -g bower'
+    flags: ['install']
+    include: 'bower.json'
+
+  BundleInstall:
+    enabled: false
+    description: 'Installing Bundler dependencies'
+    requires_files: true
+    required_executable: 'bundle'
+    install_command: 'gem install bundler'
+    flags: ['install']
+    include:
+      - 'Gemfile'
+      - 'Gemfile.lock'
+      - '*.gemspec'
+
   IndexTags:
     enabled: false
     description: 'Generating tags file from source'
     quiet: true
     required_executable: 'ctags'
 
+  NpmInstall:
+    enabled: false
+    description: 'Installing NPM dependencies'
+    requires_files: true
+    required_executable: 'npm'
+    flags: ['install']
+    include:
+      - 'package.json'
+      - 'npm-shrinkwrap.json'
+
   SubmoduleStatus:
     enabled: false
     description: 'Checking submodule status'

data/lib/overcommit.rb

@@ -1,5 +1,7 @@
+require 'overcommit/os'
 require 'overcommit/constants'
 require 'overcommit/exceptions'
+require 'overcommit/utils/file_utils'
 require 'overcommit/utils'
 require 'overcommit/git_version'
 require 'overcommit/configuration_validator'

data/lib/overcommit/cli.rb

@@ -1,3 +1,4 @@
+require 'overcommit'
 require 'optparse'
 
 module Overcommit

data/lib/overcommit/configuration.rb

@@ -71,7 +71,7 @@ module Overcommit
       Overcommit::Utils.supported_hook_types.each do |hook_type|
         hook_type_class_name = Overcommit::Utils.camel_case(hook_type)
 
-        directory = File.join(plugin_directory, hook_type.gsub('-', '_'))
+        directory = File.join(plugin_directory, hook_type.tr('-', '_'))
         plugin_paths = Dir[File.join(directory, '*.rb')].sort
 
         hook_names = plugin_paths.map do |path|

data/lib/overcommit/git_repo.rb

@@ -83,8 +83,7 @@ module Overcommit
       lines
     end
 
-    # Returns the names of all files that have been modified from compared to
-    # HEAD.
+    # Returns the names of all files that have been modified compared to HEAD.
     #
     # @param options [Hash]
     # @return [Array<String>] list of absolute file paths

data/lib/overcommit/hook/base.rb

@@ -147,7 +147,7 @@ module Overcommit::Hook
     # Gets a list of staged files that apply to this hook based on its
     # configured `include` and `exclude` lists.
     def applicable_files
-      @applicable_files ||= modified_files.select { |file| applicable_file?(file) }
+      @applicable_files ||= modified_files.select { |file| applicable_file?(file) }.sort
     end
 
     private

data/lib/overcommit/hook/commit_msg/gerrit_change_id.rb

@@ -11,7 +11,7 @@ module Overcommit::Hook::CommitMsg
     SCRIPT_LOCATION = Overcommit::Utils.script_path('gerrit-change-id')
 
     def run
-      result = execute([SCRIPT_LOCATION, commit_message_file])
+      result = execute(['sh', SCRIPT_LOCATION, commit_message_file])
       return :pass if result.success?
 
       [:fail, result.stdout]

data/lib/overcommit/hook/post_checkout/bower_install.rb

@@ -0,0 +1,11 @@
+require 'overcommit/hook/shared/bower_install'
+
+module Overcommit::Hook::PostCheckout
+  # Runs `bower install` when a change is detected in the repository's
+  # dependencies.
+  #
+  # @see {Overcommit::Hook::Shared::BowerInstall}
+  class BowerInstall < Base
+    include Overcommit::Hook::Shared::BowerInstall
+  end
+end

data/lib/overcommit/hook/post_checkout/bundle_install.rb

@@ -0,0 +1,11 @@
+require 'overcommit/hook/shared/bundle_install'
+
+module Overcommit::Hook::PostCheckout
+  # Runs `bundle install` when a change is detected in the repository's
+  # dependencies.
+  #
+  # @see {Overcommit::Hook::Shared::BundleInstall}
+  class BundleInstall < Base
+    include Overcommit::Hook::Shared::BundleInstall
+  end
+end

data/lib/overcommit/hook/post_checkout/npm_install.rb

@@ -0,0 +1,11 @@
+require 'overcommit/hook/shared/npm_install'
+
+module Overcommit::Hook::PostCheckout
+  # Runs `npm install` when a change is detected in the repository's
+  # dependencies.
+  #
+  # @see {Overcommit::Hook::Shared::NpmInstall}
+  class NpmInstall < Base
+    include Overcommit::Hook::Shared::NpmInstall
+  end
+end

data/lib/overcommit/hook/post_commit/bower_install.rb

@@ -0,0 +1,11 @@
+require 'overcommit/hook/shared/bower_install'
+
+module Overcommit::Hook::PostCommit
+  # Runs `bower install` when a change is detected in the repository's
+  # dependencies.
+  #
+  # @see {Overcommit::Hook::Shared::BowerInstall}
+  class BowerInstall < Base
+    include Overcommit::Hook::Shared::BowerInstall
+  end
+end

data/lib/overcommit/hook/post_commit/bundle_install.rb

@@ -0,0 +1,11 @@
+require 'overcommit/hook/shared/bundle_install'
+
+module Overcommit::Hook::PostCommit
+  # Runs `bundle install` when a change is detected in the repository's
+  # dependencies.
+  #
+  # @see {Overcommit::Hook::Shared::BundleInstall}
+  class BundleInstall < Base
+    include Overcommit::Hook::Shared::BundleInstall
+  end
+end

data/lib/overcommit/hook/post_commit/npm_install.rb

@@ -0,0 +1,11 @@
+require 'overcommit/hook/shared/npm_install'
+
+module Overcommit::Hook::PostCommit
+  # Runs `npm install` when a change is detected in the repository's
+  # dependencies.
+  #
+  # @see {Overcommit::Hook::Shared::NpmInstall}
+  class NpmInstall < Base
+    include Overcommit::Hook::Shared::NpmInstall
+  end
+end

data/lib/overcommit/hook/post_merge/bower_install.rb

@@ -0,0 +1,11 @@
+require 'overcommit/hook/shared/bower_install'
+
+module Overcommit::Hook::PostMerge
+  # Runs `bower install` when a change is detected in the repository's
+  # dependencies.
+  #
+  # @see {Overcommit::Hook::Shared::BowerInstall}
+  class BowerInstall < Base
+    include Overcommit::Hook::Shared::BowerInstall
+  end
+end

data/lib/overcommit/hook/post_merge/bundle_install.rb

@@ -0,0 +1,11 @@
+require 'overcommit/hook/shared/bundle_install'
+
+module Overcommit::Hook::PostMerge
+  # Runs `bundle install` when a change is detected in the repository's
+  # dependencies.
+  #
+  # @see {Overcommit::Hook::Shared::BundleInstall}
+  class BundleInstall < Base
+    include Overcommit::Hook::Shared::BundleInstall
+  end
+end

data/lib/overcommit/hook/post_merge/npm_install.rb

@@ -0,0 +1,11 @@
+require 'overcommit/hook/shared/npm_install'
+
+module Overcommit::Hook::PostMerge
+  # Runs `npm install` when a change is detected in the repository's
+  # dependencies.
+  #
+  # @see {Overcommit::Hook::Shared::NpmInstall}
+  class NpmInstall < Base
+    include Overcommit::Hook::Shared::NpmInstall
+  end
+end

data/lib/overcommit/hook/post_rewrite/bower_install.rb

@@ -0,0 +1,11 @@
+require 'overcommit/hook/shared/bower_install'
+
+module Overcommit::Hook::PostRewrite
+  # Runs `bower install` when a change is detected in the repository's
+  # dependencies.
+  #
+  # @see {Overcommit::Hook::Shared::BowerInstall}
+  class BowerInstall < Base
+    include Overcommit::Hook::Shared::BowerInstall
+  end
+end