-
Notifications
You must be signed in to change notification settings - Fork 858
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
Bundle an ESM build of the module #608
Comments
What does this mean for the current use of |
I recommend that CommonJS modules should be phased out in favour of ES6 standards... this will allow support into Deno as well. |
Is there an update on this? Struggling to use Natural with Vite and it requires a polyfill to use Natural in the browser. |
Would love to see ES6 support |
And can the two module systems be offered together? |
I was trying to get this working last night but I think, at least with the current setup in webpack, this isn't possible. Would have to swap to rollup or esbuild (unless I'm missing something). I'd be happy to help out on this. I'm wanting to use this package in Deno Deploy, but it doesn't support npm specifiers atm which is how this package works in Deno at all. |
See #744 for first concept of ESM build with rollup. |
For better tree-shaking and to keep up with latest standards, it would be nice if this package was also published in ESM together with CommonJS for the time being. (:
The text was updated successfully, but these errors were encountered: