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
refactor: add support for TypeScript 4.0 #20368
Conversation
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. Nice that we were able to get rid of the duplication for both the mixins and accessors. A few minor comments
Thanks @devversion & @jelbourn for the input and feedback. I’ll be doing the changes tomorrow. |
@devversion & @jelbourn PTAL. Thanks. |
With this change we refactor the code to support compilation using TS 4.0. The main changes are: 1) Refactor tests to not delete non optional properties 2) Refactor components not to override properties with accessors and vice-versa which is now not permitted. https://devblogs.microsoft.com/typescript/announcing-typescript-4-0-beta/#breaking-changes
… to 10.1.0-next.8
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, I'm running a google presubmit now
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.
Still LGTM from my side. Thanks Alan!
With this change we refactor the code to support compilation using TS 4.0. 1) Refactor tests to not delete non optional properties 2) Refactor components not to override properties with accessors and vice-versa which is now not permitted. https://devblogs.microsoft.com/typescript/announcing-typescript-4-0-beta/#breaking-changes 3) update typescript to 4.0, tslint to 6.1.3 and angular packages to 10.1.0-next.8 4) add ts-compat integration test for TypeScript 3.9
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. |
With this change we refactor the code to support compilation using TS 4.0.
The main changes are: