-
-
Notifications
You must be signed in to change notification settings - Fork 225
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Update benchmarks #659
Update benchmarks #659
Conversation
README.md
Outdated
| **babelMinify** | 97.23kB | 62% | 36.09kB | 49% | 6.49 | 4996.68 | | ||
| **uglify** | 96.95kB | 63% | 35.99kB | 49% | 5.80 | 1478.59 | | ||
| **closureCompiler** | 95.29kB | 63% | 35.8kB | 50% | 8.97 | 3111.51 | | ||
| **closureCompilerJs** | 98.53kB | 62% | 36.73kB | 48% | 5.87 | 9386.95 | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what happened to butternut result?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The result is wrong. Rich-Harris/butternut#162
README.md
Outdated
| **butternut** | 39.2kB | 71% | 12.81kB | 59% | 2.88 | 281.72 | | ||
| minifier | output raw | raw win | gzip output | gzip win | parse time (ms) | minify time (ms) | | ||
| --------------------- | ----------- | ------- | ----------- | -------- | --------------- | ---------------- | | ||
| **babelMinify** | 40.63kB | 71% | 13.33kB | 60% | 2.40 | 1879.97 | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldnt it be listed as babel-minify
? Would look better and correspond to the actual name of the package
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
51efe40
to
c876932
Compare
No description provided.