Skip to content

Commit f518e87

Browse files
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
1 parent a7ec67f commit f518e87

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

tox.ini

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,10 @@ commands =
1616
pytest gitlab/tests {posargs}
1717

1818
[testenv:pep8]
19+
basepython = python3
20+
deps = -r{toxinidir}/requirements.txt
21+
-r{toxinidir}/test-requirements.txt
22+
flake8
1923
commands =
2024
flake8 {posargs} gitlab/
2125

@@ -48,7 +52,7 @@ commands = {posargs}
4852
[flake8]
4953
exclude = .git,.venv,.tox,dist,doc,*egg,build,
5054
max-line-length = 88
51-
ignore = H501,H803
55+
ignore = E501,H501,H803,W503
5256

5357
[testenv:docs]
5458
deps = -r{toxinidir}/rtd-requirements.txt

0 commit comments

Comments
 (0)