Skip to content

Commit 8890e84

Browse files
authored
Fix the collector job (#239)
This adds the missing `if: always()` without which branch protection will allow auto-merge to proceed when dependencies are cancelled. This is the same fix as in: EliahKagan/subaudit#81
1 parent 884f0a0 commit 8890e84

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

.github/workflows/test.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -247,6 +247,9 @@ jobs:
247247

248248
runs-on: ubuntu-latest
249249

250+
# Don't skip `all-pass` on cancellation, since a skipped required check won't block auto-merge.
251+
if: always()
252+
250253
steps:
251254
- name: Some failed
252255
if: contains(needs.*.result, 'cancelled') || contains(needs.*.result, 'failure')

0 commit comments

Comments
 (0)