Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps): update dependency @stencil/core to v4.19.0 #29666

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

brandyscarney
Copy link
Member

Release Notes

ionic-team/stencil (@​stencil/core)

v4.19.0

Compare Source

Bug Fixes

Features

  • cli: support generation of sass and less files (#5857) (1883812), closes #2155
  • compiler: generate export maps on build (#5809) (b6d2404)
  • complier: support type import aliasing (#5836) (7ffb25d), closes #2335
  • runtime: support declarative shadow DOM (#5792) (c837063), closes #4010
  • testing: add toHaveLastReceivedEventDetail event spy matcher (#5829) (63491de), closes #2488
  • testing: allow to disable network error logging via 'logFailingNetworkRequests' option (#5839) (dac3e33), closes #2572
  • testing: expose captureBeyondViewport in pageCompareScreenshot (#5828) (cf6a450), closes #3188

@brandyscarney brandyscarney requested a review from a team as a code owner June 26, 2024 20:52
Copy link

vercel bot commented Jun 26, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ionic-framework ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 26, 2024 8:52pm

Copy link
Contributor

@thetaPC thetaPC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@brandyscarney brandyscarney added this pull request to the merge queue Jun 26, 2024
Merged via the queue into main with commit 52ff050 Jun 26, 2024
48 checks passed
@brandyscarney brandyscarney deleted the chore-update-stencil branch June 26, 2024 21:20
brandyscarney added a commit that referenced this pull request Jun 27, 2024
…e to 4.19.1 in core (#29670)

Issue number: internal

---------

This does a couple things:

**1 - Adds a `sync` command to get the latest `core` build in
`angular-server`**

The release process for `angular-server` failed
[here](https://github.com/ionic-team/ionic-framework/actions/runs/9686982182/job/26730689874).
This failure was only made apparent because the Lerna command
[here](https://github.com/ionic-team/ionic-framework/blob/52ff0505e86dc204cb2fd2fdaf67229e6eeb36f8/.github/workflows/actions/publish-npm/action.yml#L35)
runs prior to building each package.

This should have been caught by CI on the [update to @stencil/core to
v4.19.0](#29666), but
the `angular-server` package is the only package that doesn't sync
`core` before it builds. This PR adds a `sync` command so that
`angular-server` will use the latest core build during the normal build
action.

**2 - Resolving types errors in `angular-server`**

After properly syncing `core` in `angular-server` using the command
added, running build fails with the following:

<img width="400px" alt="Screenshot 2024-06-27 at 1 16 15 PM"
src="https://github.com/ionic-team/ionic-framework/assets/6577830/68d74750-821b-4776-b563-124d8fa06a79">

This was a regression in Stencil. `@stencil/core` has been updated to
resolve these errors so the build passes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment