Don’t run pylint as precommit hook because it is slow

ypid · Mar 2, 2017 · f4ecc8b · f4ecc8b
1 parent e034500
commit f4ecc8b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/Makefile b/Makefile
@@ -39,6 +39,13 @@ remove-pre-commit-hook:
 ## }}}
 
 ## check {{{
+.PHONY: check-quick
+check-quick: check-unit-tests check-docs check-lint-quick
+
+## Subset of `check`
+.PHONY: check-precommit-hook
+check-precommit-hook: check-unit-tests-with-coverage check-integration-tests check-docs check-lint-quick
+
 .PHONY: check
 check: check-unit-tests-with-coverage check-integration-tests check-docs check-lint
 
@@ -50,8 +57,11 @@ check-tox:
 check-docs:
 	$(MAKE) "docs" > /dev/null
 
+.PHONY: check-lint-quick
+check-lint-quick: check-flake8 check-travis.yml
+
 .PHONY: check-lint
-check-lint: check-flake8 check-pylint check-pylint-tests check-travis.yml
+check-lint: check-lint-quick check-pylint check-pylint-tests
 
 .PHONY: check-flake8
 check-flake8:

diff --git a/dev/hooks/pre-commit b/dev/hooks/pre-commit
@@ -1,3 +1,3 @@
 #!/bin/bash
 
-make check
+make check-precommit-hook