-
Notifications
You must be signed in to change notification settings - Fork 294
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
ES6 modules #170
Comments
I have published a NPM package that is a modification of this to only handle es6 |
@dchest any progress on this issue? |
@evbogue not yet, sorry. I hope to work on it some time soon though. |
@dchest would love to see this one pushed through. Am willing to help, so if there is anything I can do please let me know |
@roopakv thank you! I’ll try to get some time to work on it this weekend. |
If you end up tackling this, it would be great to also get https://github.com/dchest/nacl-stream-js with native esm exports. Happy to help out there once you set an example of how you want this tackled. If you are sticking with using npm for publishing native semantically versioned esm, unpkg in ?module mode can provide browser and deno compatibility. |
FYI, we've published an interim module similar to @hakanols except supporting dual imports (using the https://www.npmjs.com/package/@dashkite/tweetnacl |
Check out my fork bmdelacruz@5c4460f! It's completely tested and (somewhat) backwards compatible. The only breaking change is that consumers now need to call one of the new functions called |
What's the minimal amount of changes needed to support ES6 module-style export?
The text was updated successfully, but these errors were encountered: