Skip to content

Commit

Permalink
maint: remove pytest-flake8
Browse files Browse the repository at this point in the history
pytest-flake8 has been broken for over a year with flake8 >= 5.0
[1][2][3], and has not had any commits since March 2022.  Drop in
favor of simply invoking flake8 separately outside of pytest.  Ensure
that "make check" now invokes flake8.

[1] tholo/pytest-flake8#83
[2] tholo/pytest-flake8#87
[3] tholo/pytest-flake8#92

Signed-off-by: Daniel Harding <[email protected]>
  • Loading branch information
living180 committed Feb 14, 2023
1 parent e480fe1 commit 4f39b5c
Show file tree
Hide file tree
Showing 5 changed files with 3 additions and 7 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,7 @@ jobs:
pylint \
python-is-python3 \
python3-setuptools-scm \
python3-pytest \
python3-pytest-flake8
python3-pytest
# Runtime dependencies (required)
sudo apt-get install \
python3-qtpy \
Expand Down
3 changes: 1 addition & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -277,11 +277,10 @@ check::
$(FLAKE8) $(FLAKE8_FLAGS) $(flags) $(file)
$(PYLINT) $(PYLINT_FLAGS) --output-format=colorized $(flags) $(file)
else
# NOTE: flake8 is not part of "make check" because the pytest-flake8 plugin runs flake8
# checks during "make test" via pytest.
check:: all
check:: test
check:: doc
check:: flake8
check:: pylint
endif

Expand Down
2 changes: 1 addition & 1 deletion pytest.ini
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
[pytest]
norecursedirs=dist build .tox .eggs env*
addopts=--doctest-modules --flake8
addopts=--doctest-modules
doctest_optionflags=ALLOW_UNICODE ELLIPSIS
filterwarnings=
# https://github.com/pytest-dev/pytest/issues/6928
Expand Down
1 change: 0 additions & 1 deletion requirements/requirements-dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ PyYAML
## mock and pytest packages are used by the "make test" test suite.
pytest>=3.6
pytest-cov
pytest-flake8
pytest-black-multipy

## Build and install documentation using "make doc" and "make install-doc".
Expand Down
1 change: 0 additions & 1 deletion setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,6 @@ testing =
# upstream
pytest >= 3.5, !=3.7.3
pytest-checkdocs >= 1.2.3
pytest-flake8
pytest-black >= 0.3.7; \
# workaround for jaraco/skeleton#22
python_implementation != "PyPy"
Expand Down

0 comments on commit 4f39b5c

Please sign in to comment.