You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
The text was updated successfully, but these errors were encountered:
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 removebabel-polyfill
. Also, babel docs recommendbabel-runtime
be included as a dependency (not devDep). Thenbabel-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.
The text was updated successfully, but these errors were encountered: