Remove the extra condition on All blocking tests to ensure it runs even when tests are skipped.

For some reason, the current if statement also causes the blocking test to be skipped if any of the needs are skipped. We fix this by going back to `if: always()` to ensure that this is always run.

PiperOrigin-RevId: 663033872
pull/17816/head
Deanna Garcia 7 months ago committed by Copybara-Service
parent 29a024831f
commit f355d99f75
  1. 2
      .github/workflows/test_runner.yml

@ -263,4 +263,4 @@ jobs:
run: "${{ !contains(join(needs.*.result, ' '), 'failure') && !contains(join(needs.*.result, ' '), 'cancelled') }}"
# This workflow must run even if one or more of the dependent workflows
# failed.
if: always() && ${{ needs.set-vars.result != 'skipped' }}
if: always()

Loading…
Cancel
Save