Skip to content

[code-infra] Fully resolve imports in ESM target#3975

Merged
Janpot merged 2 commits intomui:masterfrom
Janpot:import-extensions
Aug 23, 2024
Merged

[code-infra] Fully resolve imports in ESM target#3975
Janpot merged 2 commits intomui:masterfrom
Janpot:import-extensions

Conversation

@Janpot
Copy link
Copy Markdown
Member

@Janpot Janpot commented Aug 21, 2024

Stepping stone towards full ESM support.
This babel plugin replaces imports with their full resolved path + extension.

./path/to/module becomes ./path/to/module.js or ./path/to/module/index.js, depending on where it resolves to.

See mui/material-ui#43294

blocked on

@Janpot Janpot added the scope: code-infra Involves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd). label Aug 21, 2024
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged. label Aug 22, 2024
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged. label Aug 22, 2024
@Janpot Janpot marked this pull request as ready for review August 22, 2024 16:33
@Janpot Janpot requested a review from a team August 22, 2024 16:33
@Janpot Janpot merged commit e4044bc into mui:master Aug 23, 2024
@Janpot Janpot deleted the import-extensions branch August 23, 2024 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: code-infra Involves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant