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
feat(core): support TypeScript 4.2 #41158
Conversation
73825ef
to
afafad5
Compare
We should wait for the TypeScript team to back port this issue microsoft/TypeScript#43053 to the 4.2.X version of TypeScript (you can track the progress here : microsoft/TypeScript#43180) The deprecation warnings given by TSLint + RxJS + TypeScript 4.2 can cause a lot of CI pipelines to break |
fe57424
to
6af2367
Compare
Related PR that updates ngcc to be able to deal with TS4.2 generated libraries: #41201. |
17acff1
to
53b02c1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, Reviewed-for: zone.js
Typically in majors we drop support of earlier TS versions. Are we going to drop support for < 4.2 in a seperate PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed-for: fw-forms
Reviewed-for: fw-common
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reviewed-for: fw-router
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reviewed-for: language-service
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as discussed offline, please increase the lower limit for typescript support to 4.2.3 and delete the 4.0 typings tests. Let's keep the 4.1 typings tests for now while g3 is being updated.
otherwise lgtm. thank you!
Reviewed-for: global-approvers
Updates the repo to TypeScript 4.2 and tslib 2.1.0.
Drops support for TypeScript 4.0 and 4.1 across the repo. The typings check for 4.1 was kept in order to ensure that we don't break g3.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for the compiler.
Changes for TCBs seem to be related to ts.Printer
differences around casts, which shouldn't impact the behavior of TCBs.
FYI, presubmit is successful for the changes in this PR. |
Drops support for TypeScript 4.0 and 4.1 across the repo. The typings check for 4.1 was kept in order to ensure that we don't break g3. PR Close #41158
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Updates the repo to TypeScript 4.2 and tslib 2.1.0.