diff options
author | Andrew Morgan <1342360+anoadragon453@users.noreply.github.com> | 2021-01-21 18:26:52 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-01-21 18:26:52 +0000 |
commit | 31c5382d7a439146b9e72590b634b1772b97a6f1 (patch) | |
tree | fe53c5691f1a8a8a3999881c8f1b610263d6f530 | |
parent | Speed up chain cover calculation (#9176) (diff) | |
download | synapse-31c5382d7a439146b9e72590b634b1772b97a6f1.tar.xz |
Align the directories linted in CI with the defaults in scripts-dev/lint.sh (#9191)
The lists of source directories to lint between `tox.ini` and `lint.sh` became out of sync. This PR tightens them up and adds some comments reminding any future readers to keep the list in sync.
-rw-r--r-- | changelog.d/9191.misc | 1 | ||||
-rwxr-xr-x | scripts-dev/lint.sh | 3 | ||||
-rw-r--r-- | tox.ini | 3 |
3 files changed, 5 insertions, 2 deletions
diff --git a/changelog.d/9191.misc b/changelog.d/9191.misc new file mode 100644 index 0000000000..b4bc6be13a --- /dev/null +++ b/changelog.d/9191.misc @@ -0,0 +1 @@ +Add some missing source directories to the automatic linting script. \ No newline at end of file diff --git a/scripts-dev/lint.sh b/scripts-dev/lint.sh index f328ab57d5..fe2965cd36 100755 --- a/scripts-dev/lint.sh +++ b/scripts-dev/lint.sh @@ -80,7 +80,8 @@ else # then lint everything! if [[ -z ${files+x} ]]; then # Lint all source code files and directories - files=("synapse" "tests" "scripts-dev" "scripts" "contrib" "synctl" "setup.py" "synmark") + # Note: this list aims the mirror the one in tox.ini + files=("synapse" "docker" "tests" "scripts-dev" "scripts" "contrib" "synctl" "setup.py" "synmark" "stubs" ".buildkite") fi fi diff --git a/tox.ini b/tox.ini index 5210e7b860..1a3489344f 100644 --- a/tox.ini +++ b/tox.ini @@ -24,7 +24,8 @@ deps = # install the "enum34" dependency of cryptography. pip>=10 -# directories/files we run the linters on +# directories/files we run the linters on. +# if you update this list, make sure to do the same in scripts-dev/lint.sh lint_targets = setup.py synapse |