diff options
author | Florian Bruhin <git@the-compiler.org> | 2017-12-15 23:20:09 +0100 |
---|---|---|
committer | Florian Bruhin <git@the-compiler.org> | 2017-12-15 23:22:26 +0100 |
commit | b8a5c04b69b2d5d4ac7989becb962dd5d134985f (patch) | |
tree | c405e6ab13471eedbfe202aba7554d37ae9a43f9 | |
parent | c506ffa4cd07db6326280d11340963da64fca05e (diff) | |
download | qutebrowser-b8a5c04b69b2d5d4ac7989becb962dd5d134985f.tar.gz qutebrowser-b8a5c04b69b2d5d4ac7989becb962dd5d134985f.zip |
Clean up run_pylint_on_tests
-rw-r--r-- | scripts/dev/run_pylint_on_tests.py | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/scripts/dev/run_pylint_on_tests.py b/scripts/dev/run_pylint_on_tests.py index 198d1e0e6..3dba0318d 100644 --- a/scripts/dev/run_pylint_on_tests.py +++ b/scripts/dev/run_pylint_on_tests.py @@ -49,12 +49,12 @@ def main(): files.append(os.path.join(dirpath, fn)) disabled = [ + # pytest fixtures 'redefined-outer-name', 'unused-argument', + # things which are okay in tests 'missing-docstring', 'protected-access', - # https://bitbucket.org/logilab/pylint/issue/511/ - #'undefined-variable', 'len-as-condition', # directories without __init__.py... 'import-error', @@ -65,9 +65,7 @@ def main(): toxinidir, ] - no_docstring_rgx = ['^__.*__$', '^setup$'] args = (['--disable={}'.format(','.join(disabled)), - '--no-docstring-rgx=({})'.format('|'.join(no_docstring_rgx)), '--ignored-modules=helpers,pytest,PyQt5'] + sys.argv[2:] + files) env = os.environ.copy() |