Skip to content

Commit e55dec9

Browse files
committed
CI: adjust the required job to never be skipped.
While this fails a normal required status check, the merge group will succeed even if a required status check is skipped.
1 parent 8796013 commit e55dec9

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

.github/workflows/main.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -83,14 +83,14 @@ jobs:
8383
name: docs
8484
path: docs/_build
8585

86-
required: # group all required workflows into one for the required status check
86+
required: # group all required workflows into one to avoid reconfiguring this in Actions settings
8787
needs:
8888
- test
8989
- document
90+
if: always() && !contains(needs.*.result, 'cancelled')
9091
runs-on: ubuntu-latest
9192
steps:
92-
- run: |
93-
true
93+
- run: ${{ contains(needs.*.result, 'failure') && 'false' || 'true' }}
9494

9595
publish-docs:
9696
needs: document

0 commit comments

Comments
 (0)