From f518e87b5492f2f3c201d4d723c07c746a385b6e Mon Sep 17 00:00:00 2001 From: "John L. Villalovos" Date: Tue, 23 Feb 2021 16:40:17 -0800 Subject: [PATCH] fix: tox pep8 target, so that it can run Previously running the pep8 target would fail as flake8 was not installed. Now install flake8 for the pep8 target. NOTE: Running the pep8 target fails as there are many warnings/errors. But it does allow us to run it and possibly work on reducing these warnings/errors in the future. In addition, add two checks to the ignore list as black takes care of formatting. The two checks added to the ignore list are: * E501: line too long * W503: line break before binary operator --- tox.ini | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tox.ini b/tox.ini index 826e08128..f45e74265 100644 --- a/tox.ini +++ b/tox.ini @@ -16,6 +16,10 @@ commands = pytest gitlab/tests {posargs} [testenv:pep8] +basepython = python3 +deps = -r{toxinidir}/requirements.txt + -r{toxinidir}/test-requirements.txt + flake8 commands = flake8 {posargs} gitlab/ @@ -48,7 +52,7 @@ commands = {posargs} [flake8] exclude = .git,.venv,.tox,dist,doc,*egg,build, max-line-length = 88 -ignore = H501,H803 +ignore = E501,H501,H803,W503 [testenv:docs] deps = -r{toxinidir}/rtd-requirements.txt