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

switch to ES6-compatible minifier #1148

Merged
merged 1 commit into from Jun 25, 2017
Merged

switch to ES6-compatible minifier #1148

merged 1 commit into from Jun 25, 2017

Conversation

@feross
Copy link
Member

feross commented Jun 23, 2017

Some downstream dependencies have introduced ES6 (bencode@1.0.0) and
there are others that have done so too, but which I've refrained from
updating (junk@2.0.0).

It's going to keep getting harder and harder to keep this code out of
the codebase -- so let's use babili, which is a minifier that supports
ES6.

It compresses worse than uglify (I think the bundle is something like
3KB larger now), but it's worth it IMO.

Some downstream dependencies have introduced ES6 (bencode@1.0.0) and
there are others that have done so too, but which I've refrained from
updating (junk@2.0.0).

It's going to keep getting harder and harder to keep this code out of
the codebase -- so let's use babili, which is a minifier that supports
ES6.

It compresses worse than uglify (I think the bundle is something like
3KB larger now), but it's worth it IMO.
Copy link
Member

DiegoRBaquero left a comment

I've been using babili for a while and I find it reliable. The minor size increase is acceptable.

@DiegoRBaquero DiegoRBaquero merged commit ff00a54 into master Jun 25, 2017
5 checks passed
5 checks passed
Node Security No known vulnerabilities found
Details
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@DiegoRBaquero DiegoRBaquero deleted the es6 branch Sep 28, 2017
@lock lock bot locked as resolved and limited conversation to collaborators May 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.