Run lints under poetry in CI; remove lint tox jobs (#12434)
Co-authored-by: Dan Callahan <danc@element.io>
1 files changed, 1 insertions, 30 deletions
diff --git a/tox.ini b/tox.ini
index b4ce400edf..4cd9dfb966 100644
--- a/tox.ini
+++ b/tox.ini
@@ -1,5 +1,5 @@
[tox]
-envlist = py37, py38, py39, py310, check_codestyle, check_isort
+envlist = py37, py38, py39, py310
# we require tox>=2.3.2 for the fix to https://github.com/tox-dev/tox/issues/208
minversion = 2.3.2
@@ -32,20 +32,6 @@ deps =
# install the "enum34" dependency of cryptography.
pip>=10
-# directories/files we run the linters on.
-# TODO: this is now out of date; we will remove as part of poetry migration.
-lint_targets =
- setup.py
- synapse
- tests
- # annoyingly, black doesn't find these so we have to list them
- scripts-dev
- stubs
- contrib
- synmark
- .ci
- docker
-
# default settings for all tox environments
[testenv]
deps =
@@ -116,18 +102,3 @@ setenv =
commands =
python -m synmark {posargs:}
-[testenv:check_codestyle]
-extras = lint
-commands =
- python -m black --check --diff {[base]lint_targets}
- flake8 {[base]lint_targets} {env:PEP8SUFFIX:}
-
-[testenv:check_isort]
-extras = lint
-commands = isort -c --df {[base]lint_targets}
-
-[testenv:mypy]
-deps =
- {[base]deps}
-extras = all,mypy
-commands = mypy
|