NotificationsYou must be signed in to change notification settings
Fork673
Star2.4k

Commitf518e87

committed

fix: tox pep8 target, so that it can run

Previously running the pep8 target would fail as flake8 was notinstalled.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 thesewarnings/errors in the future.In addition, add two checks to the ignore list as black takes care offormatting. The two checks added to the ignore list are: * E501: line too long * W503: line break before binary operator

1 parenta7ec67f commitf518e87Copy full SHA for f518e87

File tree

1 file changed

-1

lines changed

tox.ini

1 file changed

-1

lines changed

`‎tox.ini`

Lines changed: 5 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -16,6 +16,10 @@ commands =`
`16`	`16`	`pytest gitlab/tests {posargs}`
`17`	`17`
`18`	`18`	`[testenv:pep8]`
	`19`	`+basepython = python3`
	`20`	`+deps = -r{toxinidir}/requirements.txt`
	`21`	`+ -r{toxinidir}/test-requirements.txt`
	`22`	`+ flake8`
`19`	`23`	`commands =`
`20`	`24`	`flake8 {posargs} gitlab/`
`21`	`25`
`@@ -48,7 +52,7 @@ commands = {posargs}`
`48`	`52`	`[flake8]`
`49`	`53`	`exclude = .git,.venv,.tox,dist,doc,*egg,build,`
`50`	`54`	`max-line-length = 88`
`51`		`-ignore = H501,H803`
	`55`	`+ignore =E501,H501,H803,W503`
`52`	`56`
`53`	`57`	`[testenv:docs]`
`54`	`58`	`deps = -r{toxinidir}/rtd-requirements.txt`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitf518e87

File tree

1 file changed

1 file changed

`‎tox.ini`

0 commit comments