Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Commit

Permalink
Align the directories linted in CI with the defaults in scripts-dev/l…
Browse files Browse the repository at this point in the history
…int.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.
  • Loading branch information
anoadragon453 authored Jan 21, 2021
1 parent 758ed5f commit 31c5382
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
1 change: 1 addition & 0 deletions changelog.d/9191.misc
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Add some missing source directories to the automatic linting script.
3 changes: 2 additions & 1 deletion scripts-dev/lint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -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

Expand Down
3 changes: 2 additions & 1 deletion tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit 31c5382

Please sign in to comment.