Infra
/
lemur
5
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix code reference to older version of flake8 (#426) 

The pre-commit module contained a reference to the variable
DEFAULT_CONFIG which does not exist in the version of flake8 packaged
with the project. The DEFAULT_CONFIG defines the path to the current
user's local flake8 config file.

The flake8 2.6.2 version packaged with project has been refactored to
set the local flake8 config file in the flake8/main.py module.
This commit is contained in:
Charles Hendrie 2016-09-18 13:05:29 -05:00 committed by kevgliss
parent 39645a1a84
commit 3ce87c8a6b
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
hooks/pre-commit
View File

@ -17,13 +17,12 @@ if 'VIRTUAL_ENV' in os.environ:
def py_lint(files_modified): def py_lint(files_modified):
from flake8.main import DEFAULT_CONFIG
from flake8.engine import get_style_guide from flake8.engine import get_style_guide
# remove non-py files and files which no longer exist # remove non-py files and files which no longer exist
files_modified = filter(lambda x: x.endswith('.py'), files_modified) files_modified = filter(lambda x: x.endswith('.py'), files_modified)
flake8_style = get_style_guide(parse_argv=True, config_file=DEFAULT_CONFIG) flake8_style = get_style_guide(parse_argv=True)
report = flake8_style.check_files(files_modified) report = flake8_style.check_files(files_modified)
return report.total_errors != 0 return report.total_errors != 0