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
switch to rollup to tackle large bundle size #94
base: master
Are you sure you want to change the base?
Conversation
node js 6 & 8 build still fail , see https://travis-ci.org/github/puleos/object-hash/builds/701670409 |
Yep, |
To be honest, I would probably release this as a semver-major change anyway. You might as well drop unsupported Node.js versions from the CI matrix. |
then it be better to completely drop gulp in favor of npm scripts and rollup |
Yeah, I’m definitely okay with that too. But really, I’m just glad to accept PRs here :) |
Hi. I see that this PR is present here for some time. Any ETA on when this great improvement is going to be merged and released? |
There are still issues with this pull request. i'm unable to find the time to fix this right now, feel free to contribute. |
This would switch from browserify to rollup and only bundle neccessary code from crypto.