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
Use injectInitialization
to generate ts parameter properties
#9610
Use injectInitialization
to generate ts parameter properties
#9610
Conversation
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/10343/ |
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.
I guess we shouldn't have problems with backwards compatibility as the dependency will be correct? Sometimes this kind of things broke in the past, if people have old versions of some packages.
Since the package is a direct dependency it will always use the correct version |
Will this land? Seems like it'll fix downstream facebook/create-react-app#5868 |
This PR fixes something where Babel is tecnically correct and
babel-plugin-istanbul
is buggy, but there are an open feature request and PR to allow this behavior in TypeScript (see #9609 (comment)).I think that we can fix this in Babel.
cc @babel/typescript