Skip to content

Commit

Permalink
TST: do run full flake8 checks, use only specific ignores
Browse files Browse the repository at this point in the history
  • Loading branch information
bsipocz committed Oct 17, 2022
1 parent bae4223 commit 166bd83
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 28 deletions.
3 changes: 3 additions & 0 deletions setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,9 @@ filterwarnings =
max-line-length = 110
max-doc-length = 110
exclude = __init__.py, conf.py, setup.py, version.py, conftest.py
# W503 line break before operator
ignore = W503


[pycodestyle]
max-line-length = 110
Expand Down
30 changes: 2 additions & 28 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -70,34 +70,8 @@ commands =
sphinx-build -W -b html . _build/html

[testenv:codestyle]
# We list the warnings/errors to check for here rather than in setup.cfg because
# we don't want these options to apply whenever anyone calls flake8 from the
# command-line or their code editor - in this case all warnings/errors should be
# checked for. The warnings/errors we check for here are:
# E101 - mix of tabs and spaces
# W191 - use of tabs
# W291 - trailing whitespace
# W292 - no newline at end of file
# W293 - trailing whitespace
# W391 - blank line at end of file
# E111 - 4 spaces per indentation level
# E112 - 4 spaces per indentation level
# E113 - 4 spaces per indentation level
# E301 - expected 1 blank line, found 0
# E302 - expected 2 blank lines, found 0
# E303 - too many blank lines (3)
# E304 - blank lines found after function decorator
# E305 - expected 2 blank lines after class or function definition
# E306 - expected 1 blank line before a nested definition
# E502 - the backslash is redundant between brackets
# E722 - do not use bare except
# E901 - SyntaxError or IndentationError
# E902 - IOError
# E999: SyntaxError -- failed to compile a file into an Abstract Syntax Tree
# F822: undefined name in __all__
# F823: local variable name referenced before assignment
skip_install = true
description = check code style, e.g. with flake8
description = check code style
deps = flake8
changedir = {toxinidir}
commands = flake8 pyvo --count --select=E101,W191,W291,W292,W293,W391,E111,E112,E113,E30,E502,E722,E901,E902,E999,F822,F823
commands = flake8 pyvo --count

0 comments on commit 166bd83

Please sign in to comment.