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

Change bundle-collapser for browser-pack-flat #10

Closed
wants to merge 1 commit into from

Conversation

@casr
Copy link

@casr casr commented Dec 9, 2017

I replaced bundle-collapser with browser-pack-flat which is able to do a similar thing to Rollup's inlining but it works for CommonJS.

A sample test result from Safari 11.0.1 (note: I removed Closure from the results as it didn't build correctly for me):

100 modules

Bundler Load time (ms) Run time (ms) Total time (ms)
browserify 3 0.4 3.4
browserify-flat 2.6 0 2.6
webpack 2.3 0.5 2.8
rollup 2.2 0.1 2.3
rjs 3.5 0.1 3.6
rjs-almond 2.5 0.3 2.8

1000 modules

Bundler Load time (ms) Run time (ms) Total time (ms)
browserify 4.8 3 7.8
browserify-flat 2.5 0 2.5
webpack 3.4 2.1 5.5
rollup 2.4 0 2.4
rjs 4.5 0.3 4.8
rjs-almond 3.9 0.7 4.6

5000 modules

Bundler Load time (ms) Run time (ms) Total time (ms)
browserify 11.6 14 25.6
browserify-flat 3.9 0.2 4.1
webpack 8 6.9 14.9
rollup 3.6 0.1 3.7
rjs 12.6 1.7 14.3
@nolanlawson
Copy link
Owner

@nolanlawson nolanlawson commented May 22, 2018

Thanks very for this PR! This repo is not a benchmark I intend to maintain indefinitely, though. It served a purpose in my blog post (spurring Webpack, Browserify, and others to improve their algorithms and add tools like browser-pack-flat), but for now I would prefer to unmaintain it. I will add a note to the blog post that Browserify and Webpack have solutions to these issues now. Thanks!

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

Successfully merging this pull request may close these issues.

None yet

2 participants