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

Feature/import map support #1704

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Feature/import map support #1704

wants to merge 13 commits into from

Conversation

merryman
Copy link
Member

@merryman merryman commented Feb 5, 2025

Adds support for defining dependencies via import maps which is crucial to move away from jspm.dev.
Also is crucial for the import-map support, since a variety of babel related modules needed for that are currently broken on jspm.dev and wont be fixed anymore.

@merryman merryman requested a review from linusha February 5, 2025 17:05
@merryman merryman force-pushed the feature/import-map-support branch from c6bdfcc to c0b3586 Compare February 5, 2025 17:29
Previously some weirdly formatted es modules from esm servers would not be categorized correctly.
Issue that happens if a local variable in a module has the same name as its exported alias, which the transformed code previously would cause to be overridden. This only works with the babel transform for now.
@merryman merryman force-pushed the feature/import-map-support branch from c0b3586 to 565638b Compare February 6, 2025 15:23
Previously would return the wrong expression for export {} statements that were directly adjacent to another object expression.
Previously we would consider if/else blocks to be of the same scope as the aprent, leading to incorrect transformations in case of let and const statements.
@merryman merryman force-pushed the feature/import-map-support branch from 565638b to 5d954c3 Compare February 6, 2025 15:51
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.

1 participant