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

Push binding if importClause is default export #1528

Closed
wants to merge 6 commits into from
Closed

Push binding if importClause is default export #1528

wants to merge 6 commits into from

Conversation

IIFelix
Copy link
Collaborator

@IIFelix IIFelix commented Jun 30, 2022

Fixes #1524

If importClause is default export binding won't be added.
@johnsoncodehk pls review because i dont know if this break anything. Tests are running fine.

@IIFelix IIFelix changed the title Push binding if importClause if default export Push binding if importClause is default export Jul 1, 2022
@johnsoncodehk
Copy link
Member

johnsoncodehk commented Jul 7, 2022

This problem is tricky 🤦, maybe have to big change to resolve. (btw admire you found the code, I know how that hard to read the code. :))

@IIFelix
Copy link
Collaborator Author

IIFelix commented Jul 7, 2022

Thanks 😄

This pull request was closed.
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.

Cannot resolve the typing of default export, but named export can.
2 participants