-
Notifications
You must be signed in to change notification settings - Fork 156
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
latest upgrade is broken with nodejs #174
Comments
about 95% in nodejs/server is still using commonjs. |
I totally agree, you should keep library commonjs compatible as well. It is a good example of the situation when changes are done without considering community need, instead just because of "fancy" changes. |
To be a little more specific: (for simple folk like me ;)) Moving back to V4.6.3 rectifies this runtime error. |
crazy, over update everything (and break) and nodejs is basically commonjs in 99%, so non-considerate |
If anyone wants to change the package, feel free to create a PR 😃 |
Been trying to become "less simple" - Drinking from the Firehouse. So a PR attempt might have success. So Far: |
It doesn't work out of the box with a standard react-app, I'm curious why more folks haven't run into this. |
https://the-guild.dev/blog/support-nodejs-esm
The text was updated successfully, but these errors were encountered: