diff options
| author | Richard van der Hoff <1389908+richvdh@users.noreply.github.com> | 2021-01-08 14:08:44 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-01-08 14:08:44 +0000 |
| commit | 23a59d24ae34493c2e54e1601b2d3757df35095e (patch) | |
| tree | 95bdd12088abb1c0a019d43a41ddfcfb092f8565 /tox.ini | |
| parent | Allow running sendToDevice on workers (#9044) (diff) | |
| download | synapse-23a59d24ae34493c2e54e1601b2d3757df35095e.tar.xz | |
Run the linters on a consistent list of files (#9038)
We were running some linters on some files and some on others. Extract a common setting and use it everywhere.
Diffstat (limited to 'tox.ini')
| -rw-r--r-- | tox.ini | 20 |
1 files changed, 17 insertions, 3 deletions
diff --git a/tox.ini b/tox.ini index ab4ae295a9..297136fcc5 100644 --- a/tox.ini +++ b/tox.ini @@ -24,6 +24,20 @@ deps = # install the "enum34" dependency of cryptography. pip>=10 +# directories/files we run the linters on +lint_targets = + setup.py + synapse + tests + scripts + scripts-dev + stubs + contrib + synctl + synmark + .buildkite + docker + # default settings for all tox environments [testenv] deps = @@ -130,13 +144,13 @@ commands = [testenv:check_codestyle] extras = lint commands = - python -m black --check --diff . - /bin/sh -c "flake8 synapse tests scripts scripts-dev contrib synctl {env:PEP8SUFFIX:}" + python -m black --check --diff {[base]lint_targets} + flake8 {[base]lint_targets} {env:PEP8SUFFIX:} {toxinidir}/scripts-dev/config-lint.sh [testenv:check_isort] extras = lint -commands = /bin/sh -c "isort -c --df --sp setup.cfg synapse tests scripts-dev scripts" +commands = isort -c --df --sp setup.cfg {[base]lint_targets} [testenv:check-newsfragment] skip_install = True |