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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

@babel-core: parse should parse only #10914

Merged
merged 2 commits into from Dec 24, 2019
Merged

Conversation

kaicataldo
Copy link
Member

@kaicataldo kaicataldo commented Dec 23, 2019

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

This fixes the issue of ClassPrivateMethod nodes causing @babel/core.parse() to throw.

Currently, @babel/core.parse() is doing a lot more than just parsing - this change makes it so that it now just loads the config and parses with the normalized options.

This will unblock #10913.

@kaicataldo kaicataldo added i: bug pkg: core PR: Bug Fix 馃悰 A type of pull request used for our changelog categories and removed i: bug labels Dec 23, 2019
@kaicataldo kaicataldo changed the title Parse only @babel-core: parse should parse only Dec 23, 2019
export type ParseResult = AstRoot;

export type FileParseCallback = {
type FileParseCallback = {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(Note for reviewers)
This wasn't already imported anywhere.

@nicolo-ribaudo nicolo-ribaudo merged commit ee5b79d into babel:master Dec 24, 2019
@kaicataldo kaicataldo deleted the parse-only branch December 24, 2019 19:58
@kaicataldo kaicataldo mentioned this pull request Jan 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated A closed issue/PR that is archived due to age. Recommended to make a new issue pkg: core 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.

None yet

3 participants