Skip to content

Commit 2c33e0c

Browse files
authored
Fix merge queue tests-passed check logic (#3306)
1 parent 729f177 commit 2c33e0c

File tree

2 files changed

+14
-3
lines changed

2 files changed

+14
-3
lines changed

.github/workflows/merge_group.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,12 @@ jobs:
5353

5454
tests-passed:
5555
needs: [call-ci, call-hexagon, call-xtensa-presubmit, call-cortex-m, call-cortex-m-armclang, call-riscv-postmerge, call-xtensa-postmerge]
56+
if: always()
5657
runs-on: ubuntu-latest
5758
steps:
58-
- run: exit 0
59+
- run: |
60+
if [[ "${{ contains(needs.*.result, 'failure') }}" == "true" || "${{ contains(needs.*.result, 'cancelled') }}" == "true" ]]; then
61+
echo "One or more dependent jobs failed."
62+
exit 1
63+
fi
64+
exit 0

.github/workflows/tests_ci_ready.yml

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,12 @@ jobs:
7171

7272
tests-passed:
7373
needs: [gatekeeper, call-ci, call-hexagon, call-xtensa-presubmit, call-check-tflite-files]
74-
if: needs.gatekeeper.outputs.should_run == 'true'
74+
if: always()
7575
runs-on: ubuntu-latest
7676
steps:
77-
- run: exit 0
77+
- run: |
78+
if [[ "${{ contains(needs.*.result, 'failure') }}" == "true" || "${{ contains(needs.*.result, 'cancelled') }}" == "true" ]]; then
79+
echo "One or more dependent jobs failed."
80+
exit 1
81+
fi
82+
exit 0

0 commit comments

Comments
 (0)