summary refs log tree commit diff
path: root/.github
diff options
context:
space:
mode:
authorErik Johnston <erikj@element.io>2024-01-10 13:53:44 +0000
committerGitHub <noreply@github.com>2024-01-10 13:53:44 +0000
commit72e9b74bbfa7086e47f05271e5a2f86ad8408370 (patch)
treee0947a496bfd9b72bf48eef15aa564cb0ac72c0f /.github
parentRemove CI check for sign off (#16776) (diff)
downloadsynapse-72e9b74bbfa7086e47f05271e5a2f86ad8408370.tar.xz
Fix auto-merge CI to correctly wait for linting. (#16781)
Otherwise if you hit the `Enable auto-merge` button and the linting
fails the PR is still aut-merged.
Diffstat (limited to '.github')
-rw-r--r--.github/workflows/tests.yml19
1 files changed, 18 insertions, 1 deletions
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index 024d18bd0f..7ce22befce 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -282,10 +282,26 @@ jobs:
       - check-schema-delta
       - check-lockfile
       - lint-clippy
+      - lint-clippy-nightly
       - lint-rustfmt
     runs-on: ubuntu-latest
     steps:
-      - run: "true"
+      - uses: matrix-org/done-action@v2
+        with:
+          needs: ${{ toJSON(needs) }}
+
+          # Various bits are skipped if there was no applicable changes.
+          skippable: |
+            check-sampleconfig
+            check-schema-delta
+            lint
+            lint-mypy
+            lint-newsfile
+            lint-pydantic
+            lint-clippy
+            lint-clippy-nightly
+            lint-rustfmt
+
 
   calculate-test-jobs:
     if: ${{ !cancelled() && !failure() }} # Allow previous steps to be skipped, but not fail
@@ -695,6 +711,7 @@ jobs:
       - complement
       - cargo-test
       - cargo-bench
+      - linting-done
     runs-on: ubuntu-latest
     steps:
       - uses: matrix-org/done-action@v2