switch to rollup to tackle large bundle size

[jburghardt]

This would switch from browserify to rollup and only bundle neccessary code from crypto.

[jburghardt]

node js 6 & 8 build still fail , see https://travis-ci.org/github/puleos/object-hash/builds/701670409

[darscan]

node js 6 & 8 build still fail , see https://travis-ci.org/github/puleos/object-hash/builds/701670409

Yep, rollup needs Node 10+: rollup/rollup#3346 Which makes things tricky.. I guess you'd want to "build" on 10/12, but then test the output on 6, 8, 10 etc, so you'd need to change the Travis config to do that.

[addaleax]

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.

[jburghardt]

then it be better to completely drop gulp in favor of npm scripts and rollup

[addaleax]

Yeah, I’m definitely okay with that too. But really, I’m just glad to accept PRs here :)

[sheff146]

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?

[jburghardt]

There are still issues with this pull request. i'm unable to find the time to fix this right now, feel free to contribute.