diff options
author | xNinjaKittyx <xNinjaKittyx@users.noreply.github.com> | 2020-12-15 17:21:33 -0800 |
---|---|---|
committer | xNinjaKittyx <xNinjaKittyx@users.noreply.github.com> | 2020-12-15 18:20:13 -0800 |
commit | 9aa54a5b27468d61337528cb1e1b5b9b11a80978 (patch) | |
tree | 567693115cc101efb9254a96d96d80e9f9ccd557 /tasks.py | |
parent | 03c65c60b39e369958b056c5c844d36d515c8a63 (diff) | |
download | cmd2-git-ci_improvements.tar.gz |
Adds pre-commit config to run various lintersci_improvements
This ads black, isort, pyupgrade, and flake8 to pre-commit-config.yaml
There are also some small changes to travis.yml and tasks.py to reduce
some repeated configurations that should be consolidated into
setup.cfg. Most other changes are automated by the linter scripts.
Diffstat (limited to 'tasks.py')
-rw-r--r-- | tasks.py | 13 |
1 files changed, 6 insertions, 7 deletions
@@ -70,7 +70,7 @@ def pytest(context, junit=False, pty=True, base=False, isolated=False): tests_cmd = command_str + ' tests' context.run(tests_cmd, pty=pty) if isolated: - for root, dirnames, _ in os.walk(str(TASK_ROOT/'tests_isolated')): + for root, dirnames, _ in os.walk(str(TASK_ROOT / 'tests_isolated')): for dir in dirnames: if dir.startswith('test_'): context.run(command_str + ' tests_isolated/' + dir) @@ -83,7 +83,7 @@ namespace.add_task(pytest) def pytest_clean(context): """Remove pytest cache and code coverage files and directories""" # pylint: disable=unused-argument - with context.cd(str(TASK_ROOT/'tests')): + with context.cd(str(TASK_ROOT / 'tests')): dirs = ['.pytest_cache', '.cache', 'htmlcov', '.coverage'] rmrf(dirs) rmrf(dirs) @@ -132,7 +132,7 @@ namespace_clean.add_task(nox_clean, 'nox') ##### DOCS_SRCDIR = 'docs' DOCS_BUILDDIR = os.path.join('docs', '_build') -SPHINX_OPTS = '-nvWT' # Be nitpicky, verbose, and treat warnings as errors +SPHINX_OPTS = '-nvWT' # Be nitpicky, verbose, and treat warnings as errors @invoke.task() @@ -150,7 +150,7 @@ namespace.add_task(docs) def doc8(context): """Check documentation with doc8""" with context.cd(TASK_ROOT_STR): - context.run('doc8 docs --ignore-path docs/_build') + context.run('doc8 docs') namespace.add_task(doc8) @@ -170,7 +170,7 @@ namespace_clean.add_task(docs_clean, name='docs') @invoke.task() def linkcheck(context): """Check external links in Sphinx documentation for integrity.""" - with context.cd(str(TASK_ROOT/'docs')): + with context.cd(str(TASK_ROOT / 'docs')): context.run('make linkcheck', pty=True) @@ -347,8 +347,7 @@ namespace.add_task(pypi_test) def flake8(context): """Run flake8 linter and tool for style guide enforcement""" with context.cd(TASK_ROOT_STR): - context.run("flake8 --ignore=E252,W503 --max-complexity=26 --max-line-length=127 --show-source --statistics " - "--exclude=.git,__pycache__,.tox,.nox,.eggs,*.egg,.venv,.idea,.pytest_cache,.vscode,build,dist,htmlcov") + context.run("flake8") namespace.add_task(flake8) |