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

Bump TypeScript to 4.5.3 #12562

Merged
merged 1 commit into from
Dec 14, 2021
Merged

Bump TypeScript to 4.5.3 #12562

merged 1 commit into from
Dec 14, 2021

Conversation

Haprog
Copy link
Contributor

@Haprog Haprog commented Dec 13, 2021

A customer has been using this.shadowRoot?.elementFromPoint method which got broken in TS 4.4 (Vaadin 21) due to lib DOM type definition changes. In V22 with TS 4.4 it requires additional casting workaround to be able to call that method. This should be resolved by upgrading to TS 4.5 (change in TS). Related: WICG/webcomponents#661 (comment)

@CLAassistant
Copy link

CLAassistant commented Dec 13, 2021

CLA assistant check
All committers have signed the CLA.

@github-actions
Copy link

Unit Test Results

   720 files     720 suites   21m 55s ⏱️
5 648 tests 5 598 ✔️ 50 💤 0
5 658 runs  5 611 ✔️ 47 💤 0

Results for commit 2399b67.

@Haprog Haprog requested a review from caalador December 13, 2021 11:02
@Artur-
Copy link
Member

Artur- commented Dec 13, 2021

Are there incompatible changes in TS 4.5 that need to be taken into account somewhere?

@Haprog
Copy link
Contributor Author

Haprog commented Dec 13, 2021

@Artur- I'm not aware of any. Generally TS 4.5 seems to have less breaking changes compared to what 4.4 had. https://www.typescriptlang.org/docs/handbook/release-notes/typescript-4-5.html#breaking-changes

If something related to those breaks, I assume it should be easy to fix.

@Artur- Artur- merged commit c8891f9 into master Dec 14, 2021
@Artur- Artur- deleted the feat/bump-typescript-to-4.5 branch December 14, 2021 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants