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

Use injectInitialization to generate ts parameter properties #9610

Merged

Conversation

nicolo-ribaudo
Copy link
Member

Q                       A
Fixed Issues? Fixes #9609
Patch: Bug Fix? Yes
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass? Yes
Documentation PR Link
Any Dependency Changes?
License MIT

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

@nicolo-ribaudo nicolo-ribaudo added PR: Bug Fix 🐛 A type of pull request used for our changelog categories area: typescript labels Feb 28, 2019
@babel-bot
Copy link
Collaborator

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/10343/

Copy link
Member

@danez danez left a 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.

@nicolo-ribaudo
Copy link
Member Author

Since the package is a direct dependency it will always use the correct version

@SimenB
Copy link
Contributor

SimenB commented May 9, 2019

Will this land? Seems like it'll fix downstream facebook/create-react-app#5868

@nicolo-ribaudo nicolo-ribaudo added this to the v7.5.0 milestone May 21, 2019
@nicolo-ribaudo nicolo-ribaudo added PR: Ready to be Merged A pull request with already two approvals, but waiting for the next minor release and removed PR: Ready to be Merged A pull request with already two approvals, but waiting for the next minor release labels May 21, 2019
@nicolo-ribaudo nicolo-ribaudo merged commit 87fb6c4 into babel:master May 21, 2019
@nicolo-ribaudo nicolo-ribaudo deleted the ts-param-prop-late-super branch May 21, 2019 20:29
@nicolo-ribaudo nicolo-ribaudo removed this from the v7.5.0 milestone May 21, 2019
@lock lock bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Oct 3, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area: typescript outdated A closed issue/PR that is archived due to age. Recommended to make a new issue PR: Bug Fix 🐛 A type of pull request used for our changelog categories
Projects
None yet
Development

Successfully merging this pull request may close these issues.

@babel/plugin-transform-typescript can insert public property initialization before the call to super.
5 participants