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

Optimise for Tree Shaking #3

Open
1000i100 opened this issue Dec 10, 2020 · 4 comments
Open

Optimise for Tree Shaking #3

1000i100 opened this issue Dec 10, 2020 · 4 comments

Comments

@1000i100
Copy link

export each public func instead of one global to help tree shaker to do there job.

@hakanols
Copy link
Owner

Would be excellent. My decision to not do that was base on not changing the API from the original tweetnacl-js (https://github.com/dchest/tweetnacl-js) for compatibility. Perhaps it is time to change that decision.

@qgustavor
Copy link

qgustavor commented Jun 20, 2024

@hakanols
Copy link
Owner

Can you check this? https://github.com/qgustavor/tweetnacl-js

Or this? https://www.npmjs.com/package/tweetnacl-ts

Looks nice but what should I check for? Is it inspiration to editing my fork?

@qgustavor
Copy link

It can be, if those doesn't fit your needs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants