feat(eslint): adds option to disable eslint when in development

dvdzkwsk · Sep 11, 2015 · 8f8fd5d · 8f8fd5d
1 parent 77831fc
commit 8f8fd5d
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 7 deletions.
diff --git a/build/webpack/client.js b/build/webpack/client.js
@@ -37,13 +37,6 @@ const webpackConfig = {
     alias : config.get('utils_aliases')
   },
   module : {
-    preLoaders : [
-      {
-        test : /\.(js|jsx)$/,
-        loaders : ['eslint-loader'],
-        include : paths.project(config.get('dir_src'))
-      }
-    ],
     loaders : [
       {
         test : /\.(js|jsx)$/,
@@ -110,4 +103,20 @@ if (globals.__PROD__) {
   );
 }
 
+// ------------------------------------
+// Optional Configuration
+// ------------------------------------
+if (
+  !globals.__DEV__ ||
+  (globals.__DEV__ && config.get('webpack_lint_in_dev'))
+) {
+  webpackConfig.module.preLoaders = [
+    {
+      test : /\.(js|jsx)$/,
+      loaders : ['eslint-loader'],
+      include : paths.project(config.get('dir_src'))
+    }
+  ];
+}
+
 export default webpackConfig;
diff --git a/config/index.js b/config/index.js
@@ -34,6 +34,9 @@ config.set('webpack_public_path',
   `http://${config.get('server_host')}:${config.get('webpack_port')}/`
 );
 
+// build options
+config.set('webpack_lint_in_dev', true);
+
 // ------------------------------------
 // Project
 // ------------------------------------