From f355d99f75e2ec7474880036e072da103ff5e665 Mon Sep 17 00:00:00 2001 From: Deanna Garcia Date: Wed, 14 Aug 2024 13:28:33 -0700 Subject: [PATCH] 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 --- .github/workflows/test_runner.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/test_runner.yml b/.github/workflows/test_runner.yml index 9e88e95794..64bad17990 100644 --- a/.github/workflows/test_runner.yml +++ b/.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()