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

🏗 Update Preact and TypeScript dependencies #39252

Merged
merged 5 commits into from
Jul 14, 2023

Conversation

danielrozenberg
Copy link
Member

@danielrozenberg danielrozenberg commented Jul 12, 2023

This PR updates Preact to v10.16.0 (which Renovate attempted to do in #37042) and TypeScript to v4.9.5 (which Renovated attempted to do in (#36642)

These two changes are co-dependent, and cause a lot of typing issues, which I fix in this PR.

For the most part these changes are trivial, but please review the change in src/core/dom/index.js that might be more serious

@danielrozenberg danielrozenberg marked this pull request as ready for review July 12, 2023 21:21
@amp-owners-bot amp-owners-bot bot requested a review from erwinmombay July 12, 2023 21:21
@amp-owners-bot
Copy link

amp-owners-bot bot commented Jul 12, 2023

Hey @estherkim! These files were changed:

build-system/tasks/e2e/amp-driver.js

Hey @ampproject/wg-performance! These files were changed:

tsconfig.base.json

@danielrozenberg danielrozenberg requested a review from powerivq July 13, 2023 17:00
@danielrozenberg danielrozenberg merged commit afb2cf7 into ampproject:main Jul 14, 2023
@danielrozenberg danielrozenberg deleted the preact-ts-change branch July 14, 2023 13:08
eszponder pushed a commit to krzysztofequativ/amphtml that referenced this pull request Apr 22, 2024
* Bump Preact to v10.16.0

* Change how Preact's `render`/`hydrate` functions are re-export via the `preact/dom` namespace to match changes in upstream

* Bump Typescript to v4.9.5

* Fix typing issues

* Revert change in build-system/tasks/build-story-localization.js and replace it with type casting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants