PatrickJS/NG6-starter

transform-runtime vs babel-polyfill

jsvg opened this issue · 2 comments

jsvg commented

I believe these two configurations of babel are redundant.

babel-polyfill provides globals for es6 features, while the runtime transform method polyfills without using global methods. The latter method is usually preferable.

Would suggest modifying package.json to remove babel-polyfill. Also, babel docs recommend babel-runtime be included as a dependency (not devDep). Then babel-polyfill can be removed from the gulp file's paths.entry array.

This change appears to reduce build size by a small amount.

Can PR this if desired.

Actually, babel docs recommend to include babel-runtime as development dependency (devDep)

http://babeljs.io/docs/plugins/transform-runtime/#installation

@JabbyPanda
babel-runtime should be included as a dependency (not devDep) and babel-plugin-transform-runtime should be included as a dev-dependency.