Class: PreCommit
- Inherits:
-
Object
- Object
- PreCommit
- Defined in:
- lib/pre-commit/checks/jslint_check.rb,
lib/pre-commit/base.rb,
lib/pre-commit/checks.rb,
lib/pre-commit/checks/js_check.rb,
lib/pre-commit/checks/jshint_check.rb,
lib/pre-commit/checks/migration_check.rb
Overview
This (PreCommit) should be a module … it should also have a cooler name :P
Defined Under Namespace
Classes: JsCheck, JshintCheck, JslintCheck, MigrationCheck
Constant Summary collapse
- WhiteSpace =
lambda { WhiteSpaceChecker.check }
- ClosureSyntaxCheck =
lambda { compiler = "test/javascript/lib/compiler.jar" if File.exists?('public/javascripts') && (args = Utils.staged_files('public/javascripts')).size > 0 ClosureChecker.check(args.split(" ")) else true end }
- Checks =
{ :white_space => WhiteSpace, :console_log => ConsoleLog, :js_lint_all => JslintCheck.new(:all), :js_lint_new => JslintCheck.new(:new), :jshint => JshintCheck.new, :debugger => DebuggerCheck, :tabs => Tabs, :closure_syntax_check => ClosureSyntaxCheck, :merge_conflict => MergeConflict, :migrations => MigrationCheck.new, :ci => CiCheck.new }
Class Method Summary collapse
-
.checks_to_run ⇒ Object
Actually, on the deprecation note.
- .root ⇒ Object
Class Method Details
.checks_to_run ⇒ Object
Actually, on the deprecation note. This method isn’t really the problem. The problem is the default generated pre-commit hook. It shouldn’t have logic in it. The we have freedom to change the gem implementation however we want, and nobody is forced to update their pre-commit binary.
53 54 55 56 57 58 59 60 61 |
# File 'lib/pre-commit/checks.rb', line 53 def self.checks_to_run checks_to_run = `git config pre-commit.checks`.chomp.split(/,\s*/).map(&:to_sym) if checks_to_run.empty? Checks.values_at(:white_space, :console_log, :debugger, :tabs, :jshint, :migrations) else Checks.values_at(*checks_to_run) end.compact end |