Skip to content

Commit 3b1ad48

Browse files
Firestar99LegNeato
authored andcommitted
ci: ensure lint and cargo-deny succeed before merging
1 parent a26e0df commit 3b1ad48

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

.github/workflows/ci.yaml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -204,7 +204,7 @@ jobs:
204204
# to update the branch protection rules when the test matrix changes
205205
test_success:
206206
runs-on: ubuntu-24.04
207-
needs: [test, compiletest, difftest, android]
207+
needs: [test, compiletest, difftest, android, lint, cargo-deny]
208208
# 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
209209
if: ${{ always() }}
210210
steps:
@@ -215,6 +215,8 @@ jobs:
215215
[[ "${{ needs.compiletest.result }}" == "success" ]] || exit 1
216216
[[ "${{ needs.difftest.result }}" == "success" ]] || exit 1
217217
[[ "${{ needs.android.result }}" == "success" ]] || exit 1
218+
[[ "${{ needs.lint.result }}" == "success" ]] || exit 1
219+
[[ "${{ needs.cargo-deny.result }}" == "success" ]] || exit 1
218220
219221
lint:
220222
name: Lint

0 commit comments

Comments
 (0)