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

[WIP] Use Bublé instead of cursed Babel #267

Merged
merged 4 commits into from
Oct 19, 2016
Merged

Conversation

danielkcz
Copy link
Contributor

@danielkcz danielkcz commented Oct 19, 2016

Just little experiment to contradict #266 with this little unknown transpiler. First impression is that it's faster then babel and there is couple of bytes saved as well. I absolutely love that it doesn't need any configuration. Just use plugin and done.

Current babel configuration (master)
Estimating dist/stampit.umd.min.js: 4.5 KB, GZIP : 1.56 KB

Using Babel 2015 preset (@Piroro-hs)
Estimating dist/stampit.umd.min.js: 4.96 KB, GZIP : 1.63 KB

Using Buble
Estimating dist/stampit.umd.min.js: 4.5 KB, GZIP : 1.52 KB

Here I made a package of all three approaches for comparison.
stampit-dist.zip

The PR needs some cleanup, just wanted to do a quick check.

@koresar
Copy link
Member

koresar commented Oct 19, 2016

Wow. Cool! Can we kill package.json->babel too? Please :)

@danielkcz
Copy link
Contributor Author

Ok, got rid of Babel, although we still need babel-cli to actually run the build because Buble apparently doesn't have alternative to babel-node :(

@koresar
Copy link
Member

koresar commented Oct 19, 2016

Looks like this PR needs to be improved because the folder still contains the bloody .babelrc file. That's one of the issues with the react-native.

@danielkcz
Copy link
Contributor Author

Yea right, I've missed that :)

@koresar koresar merged commit 7182293 into master Oct 19, 2016
@koresar koresar deleted the use-buble-no-babel branch October 19, 2016 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants