Merge pull request #3 from pre-commit/flake8

Flake8
This commit is contained in:
Anthony Sottile 2014-04-13 17:53:50 -07:00
commit ca93f6834f
4 changed files with 9 additions and 17 deletions

View file

@ -13,14 +13,19 @@
description: Ensures that a file is either empty, or ends with one newline. description: Ensures that a file is either empty, or ends with one newline.
entry: end-of-file-fixer entry: end-of-file-fixer
language: python language: python
- id: flake8
name: Flake8
description: This hook runs flake8.
entry: flake8
language: python
- id: name-tests-test - id: name-tests-test
name: Tests should end in _test.py name: Tests should end in _test.py
description: This verifies that test files are named correctly description: This verifies that test files are named correctly
entry: name-tests-test entry: name-tests-test
language: python language: python
- id: pyflakes - id: pyflakes
name: Pyflakes name: Pyflakes (DEPRECATED, use flake8)
description: This validator runs pyflakes. description: This hook runs pyflakes. (This is deprecated, use flake8).
entry: pyflakes entry: pyflakes
language: python language: python
- id: trailing-whitespace - id: trailing-whitespace

View file

@ -2,9 +2,7 @@
# Testing requirements # Testing requirements
coverage coverage
# Fuck you ipython flake8
ipython<2.0.0
ipdb
mock mock
git+git://github.com/pre-commit/pre-commit#egg=pre-commit git+git://github.com/pre-commit/pre-commit#egg=pre-commit
pytest pytest

View file

@ -7,6 +7,7 @@ setup(
packages=find_packages('.', exclude=('tests*', 'testing*')), packages=find_packages('.', exclude=('tests*', 'testing*')),
install_requires=[ install_requires=[
'argparse', 'argparse',
'flake8',
'plumbum', 'plumbum',
'pyflakes', 'pyflakes',
'pyyaml', 'pyyaml',

View file

@ -1,12 +0,0 @@
import subprocess
from pre_commit.clientlib.validate_manifest import load_manifest
def test_all_hooks_allow_no_files():
manifest = load_manifest('hooks.yaml')
for hook in manifest:
if hook['id'] != 'pyflakes':
subprocess.check_call([hook['entry']])