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
Fix version checks in .availableHelper and transform-runtime definitions. #8659
Conversation
e8196ca
to
61ec957
Compare
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/9062/ |
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.
+1 for the 7.0.1 branch release. Maybe we could also release the regressions fixes we have already merged.
Ok, I made |
Fixes #1, Fixes #2
The comments in the code pretty much explain why this change is needed. I think this is enough to fix the issue, but it's actually a little difficult to be 100% sure since I don't entirely trust
semver
anymore.As noted by the changes in the tests, this is definitely something that we need to fix in order for things to function properly. On
7.0.0
essentially any helper will show up as available, unfortunately.An alternative fix here would be to change
transform-runtime
andexternal-helpers
to stop passing7.0.0-beta.0
as their default version, but I think the approach here is better?Since we've landed so much and might just straight to 7.1.0, I'm actually tempted to branch and release a
7.0.1
with just this applied on top, so people have more of a chance to get updated to a version with a functional.availableHelper()
. On the fence there though.