We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
vulture can scan for unused variables, params etc. We should clean up our codebase and integrate the tool.
The text was updated successfully, but these errors were encountered:
I also started a longterm branch to enhance the general linting. The current prospector.yaml:
die-on-tool-error: true test-warnings: true doc-warnings: true ignore-paths: - marmoset/virt/__init__.py uses: - flask pep257: run: true disable: - D203 pep8: run: true full: true pyroma: run: true pylint: run: true dodgy: run: true mccabe: run: true
This currently finds 172 warnings, most of them missings docstrings.
Sorry, something went wrong.
The current .prospector.yaml:
test-warnings: true doc-warnings: true ignore-paths: - marmoset/virt/ uses: - flask pep257: run: true disable: - D203 pep8: run: true full: true pyroma: run: true pylint: run: true enable: - unused-variable - invalid-name dodgy: run: true mccabe: run: true
and the recommended tox.ini:
[tox] envlist = py3{4,5}, pypy3 [testenv] deps = nose -rrequirements.txt [testenv:test] commands=nosetests [testenv:testxunit] commands=nosetests --with-xunit [testenv:lint] deps = {[testenv]deps} maxcdn astroid pylint prospector[with_everything] commands = prospector \ --die-on-tool-error [testenv:sphinx] deps = {[testenv]deps} sphinx commands = python setup.py build_sphinx
we will merge the current changes into master (except for the updated prospector to keep travis green).
Part of the needed changes are implemented in #89
No branches or pull requests
vulture can scan for unused variables, params etc. We should clean up our codebase and integrate the tool.
The text was updated successfully, but these errors were encountered: