From 1eb9d977a4cdc1704841bc36f0d41b9354be9585 Mon Sep 17 00:00:00 2001 From: firestar99 Date: Sun, 12 Oct 2025 13:55:20 +0200 Subject: [PATCH] ci: ensure lint and cargo-deny succeed before merging --- .github/workflows/ci.yaml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index d12d7fd930..5e178d6aee 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -204,7 +204,7 @@ jobs: # to update the branch protection rules when the test matrix changes test_success: runs-on: ubuntu-24.04 - needs: [test, compiletest, difftest, android] + needs: [test, compiletest, difftest, android, lint, cargo-deny] # Hack for buggy GitHub Actions behavior with skipped checks: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/collaborating-on-repositories-with-code-quality-features/troubleshooting-required-status-checks#handling-skipped-but-required-checks if: ${{ always() }} steps: @@ -215,6 +215,8 @@ jobs: [[ "${{ needs.compiletest.result }}" == "success" ]] || exit 1 [[ "${{ needs.difftest.result }}" == "success" ]] || exit 1 [[ "${{ needs.android.result }}" == "success" ]] || exit 1 + [[ "${{ needs.lint.result }}" == "success" ]] || exit 1 + [[ "${{ needs.cargo-deny.result }}" == "success" ]] || exit 1 lint: name: Lint