A powerful cross-platform UI toolkit for building native-quality iOS, Android, and Progressive Web Apps with HTML, CSS, and JavaScript.
chore(ci): fix stencil nightly job dependencies (#27298)
Issue number: N/A --------- <!-- Please refer to our contributing documentation for any questions on submitting a pull request, or let us know here if you need any help: https://ionicframework.com/docs/building/contributing --> <!-- Some docs updates need to be made in the `ionic-docs` repo, in a separate PR. See https://github.com/ionic-team/ionic-framework/blob/main/.github/CONTRIBUTING.md#modifying-documentation for details. --> <!-- Please do not submit updates to dependencies unless it fixes an issue. --> <!-- Please try to limit your pull request to one type (bugfix, feature, etc). Submit multiple pull requests if needed. --> ## What is the current behavior? <!-- Please describe the current behavior that you are modifying. --> The Stencil nightly job fails with the following error: ``` The workflow is not valid. .github/workflows/stencil-nightly.yml (Line: 101, Col: 13): Job 'test-core-screenshot-legacy' depends on unknown job 'build-core'. .github/workflows/stencil-nightly.yml (Line: 122, Col: 12): Job 'verify-screenshots-legacy' depends on job 'test-core-screenshot-legacy' which creates a cycle in the dependency graph. ``` This started to fail last night. I believe this is a result of https://github.com/ionic-team/ionic-framework/pull/27228, specifically this line https://github.com/ionic-team/ionic-framework/commit/4fe8de7df71b0d9fb8d618aca5a428129d96a04b#diff-2f087e8fac034d51c50fc9350e10ceb4034298c16dd0d4f414f79d88ebb71aa0R101 ## What is the new behavior? <!-- Please describe the behavior or changes that are being added by this PR. --> this commit updates the job dependency hierarchy for test-core-screenshot-legacy from `build-core` to `build-core-with-stencil-nightly`. the former is not a valid job name for the `stencil-nightly` workflow, likely just a copy/paste error ## Does this introduce a breaking change? - [ ] Yes - [x] No <!-- If this introduces a breaking change, please describe the impact and migration path for existing applications below. --> ## Other information To test this, I was able to kick off a version of this job with this branch: https://github.com/ionic-team/ionic-framework/actions/runs/4819613134 and saw the new/legacy screenshot tests running:  <!-- Any other information that is important to this PR such as screenshots of how the component looks before and after the change. -->
R
Ryan Waskiewicz committed
ab992b02c6ad224e51bc94582117ca7c39acc1bd
Parent: a667b3e
Committed by GitHub <noreply@github.com>
on 4/27/2023, 12:37:06 PM