Remove the "modules: false" option from Babel

The Babel option "modules: false" can't be used with the add-module-exports, which is needed for Calypso to compile. This means we can't take advantage of WebPack2 tree-shaking yet, thus adding 50KB to the JS bundle. More info: 59naga/babel-plugin-add-module-exports#39 (comment) Automattic/wp-calypso#10799 Automattic/wp-calypso#11204
Automattic · Feb 20, 2017 · 31a39b7 · 31a39b7
1 parent ccfb169
commit 31a39b7
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/webpack.config.js b/webpack.config.js
@@ -10,7 +10,7 @@ const babelSettings = {
 		[ 'env', {
 			useBuiltIns: true,
 			targets: { browsers },
-			modules: false,
+			// modules: false, // add-module-exports breaks with WebPack 2 and modules: false
 		} ],
 		'stage-1',
 		'react'