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

Fix ES2018 import() #202

Merged
merged 1 commit into from
Nov 2, 2021
Merged

Fix ES2018 import() #202

merged 1 commit into from
Nov 2, 2021

Conversation

jogibear9988
Copy link
Contributor

@jogibear9988 jogibear9988 commented Nov 2, 2021

],
"loc": {
"start": {
"type": "Program",
Copy link
Owner

Choose a reason for hiding this comment

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

Is there a place I can find this file, or is it generated locally based on the test itself?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Don't know what you mean...
The AST was different with the old implementation.... Now it is correct (I think :-) )

@lahma lahma changed the title bugfix es2018 import() - https://github.com/jquery/esprima/pull/2076 Fix ES2018 import() Nov 2, 2021
Copy link
Collaborator

@lahma lahma left a comment

Choose a reason for hiding this comment

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

👍🏻

@lahma lahma merged commit d6cc2ba into sebastienros:main Nov 2, 2021
@jogibear9988 jogibear9988 deleted the fixImport branch January 3, 2022 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants