-
Notifications
You must be signed in to change notification settings - Fork 145
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
Very large bundle size #91
Comments
@smithki PRs are welcome! This module is not very actively maintained otherwise, though. |
I’m a big proponent of |
I would be willing to make a PR, but I don’t want to overstep if this is out of scope for the maintainers. |
@smithki Generally, anything that doesn’t affect consumers of this module should be just fine |
Any progress on this? |
@jburghardt -- So sorry, things are busy for me at the moment. I don't think I'll get to this anytime soon. You're welcome to take it on. |
Bumping this issue. pull request is pending. |
PRs are still welcome. |
Any interest in an ESM version for tree-shaking support? I'd be happy to send a PR for this 😄 |
According to Bundlephobia, the bundled package size is quite large (10kb gzipped!)
If only the required APIs from
crypto
are included, the bundle size reduces to a much more reasonable 1-2kb gzipped. It would be great if thecrypto
require could be tree-shaken!The text was updated successfully, but these errors were encountered: