rnhmjoj/qutebrowser
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Add pep257 to run_checks.py

Browse Source
This commit is contained in:
Florian Bruhin 2014-02-07 19:44:56 +01:00
parent 76fc591fd2
commit e8b01b2b31
1 changed files with 27 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
run_checks.py
View File

@ -1,6 +1,6 @@
""" Run different codecheckers over a codebase. """ Run different codecheckers over a codebase.
Runs flake8, pylint and a CRLF/whitespace/conflict-checker by default. Runs flake8, pylint, pep257 and a CRLF/whitespace/conflict-checker by default.
""" """
# Copyright 2014 Florian Bruhin (The Compiler) <mail@qutebrowser.org> # Copyright 2014 Florian Bruhin (The Compiler) <mail@qutebrowser.org>
@ -26,6 +26,7 @@ import os
import os.path import os.path
from collections import OrderedDict from collections import OrderedDict
import pep257
from pkg_resources import load_entry_point, DistributionNotFound from pkg_resources import load_entry_point, DistributionNotFound
status = OrderedDict() status = OrderedDict()
@ -60,6 +61,9 @@ options = {
'flake8': [ 'flake8': [
'E241', # Multiple spaces after , 'E241', # Multiple spaces after ,
], ],
'pep257': [
'D102', # Docstring missing, will be handled by others
],
}, },
'exclude': [ 'appdirs.py' ], 'exclude': [ 'appdirs.py' ],
'other': { 'other': {
@ -69,7 +73,7 @@ options = {
} }
def run(name, args=None): def run(name, args=None):
""" Run a checker with optional args. """ Run a checker via distutils with optional args.
name -- Name of the checker/binary name -- Name of the checker/binary
args -- Option list of arguments to pass args -- Option list of arguments to pass
@ -95,6 +99,18 @@ def run(name, args=None):
status[name] = None status[name] = None
print() print()
def check_pep257(args=None):
sys.argv = ['pep257', options['target']]
if args is not None:
sys.argv += args
print("====== pep257 ======")
try:
status['pep257'] = pep257.main(*pep257.parse_options())
except Exception as e:
print('{}: {}'.format(e.__class__.__name__, e))
status[name] = None
print()
def check_line(): def check_line():
"""Checks a filetree for CRLFs, conflict markers and weird whitespace""" """Checks a filetree for CRLFs, conflict markers and weird whitespace"""
print("====== line ======") print("====== line ======")
@ -144,8 +160,17 @@ def _get_args(checker):
args += options['other']['flake8'] args += options['other']['flake8']
except KeyError: except KeyError:
pass pass
elif checker == 'pep257':
args = []
try:
args += ['--ignore=' + ','.join(options['disable']['pep257'])]
args += ['--match=(?!{}).*\.py'.format('|'.join(options['exclude']))]
args += options['other']['pep257']
except KeyError:
pass
return args return args
check_pep257(_get_args('pep257'))
for checker in ['pylint', 'flake8']: for checker in ['pylint', 'flake8']:
# FIXME what the hell is the flake8 exit status? # FIXME what the hell is the flake8 exit status?
run(checker, _get_args(checker)) run(checker, _get_args(checker))