From dbb139afa065c1b930927afc26cc4ce285d9172d Mon Sep 17 00:00:00 2001 From: Ryan Waskiewicz Date: Fri, 28 Apr 2023 09:13:46 -0400 Subject: [PATCH] chore(ci): fix stencil nightly legacy check (#27317) Issue number: # --------- ## What is the current behavior? the legacy screenshot test verification step can fail due to improper configuration ## What is the new behavior? this commit updates the name of the legacy screenshot test so that we properly gate on verifying that the legacy tests passed ## Does this introduce a breaking change? - [ ] Yes - [x] No ## Other information I tested this by kicking off the workflow - https://github.com/ionic-team/ionic-framework/actions/runs/4830665737 Yesterday, I [put up a PR](https://github.com/ionic-team/ionic-framework/pull/27298) that did the same. However, I failed to verify it succeeded (since we don't gate on Stencil nightly) - only that it started (since that was the point of failure yesterday). Today, I have verified that it passes all the way through ![Screenshot 2023-04-28 at 9 12 25 AM](https://user-images.githubusercontent.com/1930213/235156949-67aa1b35-d141-4951-9f2c-c0722f11a520.png) --- .github/workflows/stencil-nightly.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/stencil-nightly.yml b/.github/workflows/stencil-nightly.yml index c1fae8048e..46b2f0732e 100644 --- a/.github/workflows/stencil-nightly.yml +++ b/.github/workflows/stencil-nightly.yml @@ -123,7 +123,7 @@ jobs: runs-on: ubuntu-latest steps: - name: Check build matrix status - if: ${{ needs.test-core-screenshot.result != 'success' }} + if: ${{ needs.test-core-screenshot-legacy.result != 'success' }} run: exit 1 build-vue: