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

ESM - Webpack 5 - Node SM - #29193

Closed
joacub opened this issue Oct 21, 2021 · 6 comments
Closed

ESM - Webpack 5 - Node SM - #29193

joacub opened this issue Oct 21, 2021 · 6 comments
Labels
status: waiting for author Issue with insufficient information

Comments

@joacub
Copy link

joacub commented Oct 21, 2021

IF we select ESM modules output, the builder fails requesting to the library to put type: module or use a mjs extension. the current configuration does not work for esm and have to be required by cjs.

@joacub joacub added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Oct 21, 2021
@mnajdova
Copy link
Member

It doesn't look like this bug report has enough info for one of us to reproduce it.

Please provide a CodeSandbox (https://material-ui.com/r/issue-template-latest), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.

Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve

@mnajdova mnajdova added status: waiting for author Issue with insufficient information and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Oct 26, 2021
@michaldudak
Copy link
Member

Closing due to inactivity.

@joacub
Copy link
Author

joacub commented Jan 7, 2022

@michaldudak
Sorry for the time to response, but this still been a problem, is impossible to use modules in node since you don't put "type": "module" so when you use esm in node it will treat like commonjs and is not

@joacub joacub changed the title ESM - Webpack 5 - Node SM ESM - Webpack 5 - Node SM - Jan 7, 2022
@michaldudak
Copy link
Member

@Janpot has already started working on it in #30510. However, as this is a breaking change, we have to wait with releasing it for the next major.

@joacub
Copy link
Author

joacub commented Jan 7, 2022

With next major you wanna say mui v6 ? Or 5.3

@michaldudak
Copy link
Member

v6 is where we'll be able to introduce breaking changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: waiting for author Issue with insufficient information
Projects
None yet
Development

No branches or pull requests

3 participants