summary refs log tree commit diff
path: root/.github/workflows
diff options
context:
space:
mode:
authorDavid Robertson <davidr@element.io>2022-04-12 17:35:48 +0100
committerGitHub <noreply@github.com>2022-04-12 17:35:48 +0100
commit58c657322a18cadf8036819ef87fce9889d65fb0 (patch)
tree83e38fb0c1f72556b05428c5f6d21419eafde8fb /.github/workflows
parentProcess device list updates asynchronously (#12365) (diff)
downloadsynapse-58c657322a18cadf8036819ef87fce9889d65fb0.tar.xz
Run lints under poetry in CI; remove lint tox jobs (#12434)
Co-authored-by: Dan Callahan <danc@element.io>
Diffstat (limited to '.github/workflows')
-rw-r--r--.github/workflows/tests.yml20
1 files changed, 7 insertions, 13 deletions
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index 5a98f61932..c300d6547b 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -20,19 +20,13 @@ jobs:
       - run: scripts-dev/config-lint.sh
 
   lint:
-    runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        toxenv:
-          - "check_codestyle"
-          - "check_isort"
-          - "mypy"
-
-    steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-python@v2
-      - run: pip install tox
-      - run: tox -e ${{ matrix.toxenv }}
+    # This does a vanilla `poetry install` - no extras. I'm slightly anxious
+    # that we might skip some typechecks on code that uses extras. However,
+    # I think the right way to fix this is to mark any extras needed for
+    # typechecking as development dependencies. To detect this, we ought to
+    # turn up mypy's strictness: disallow unknown imports and be accept fewer
+    # uses of `Any`.
+    uses: "matrix-org/backend-meta/.github/workflows/python-poetry-ci.yml@v1"
 
   lint-crlf:
     runs-on: ubuntu-latest