[Build] Do not treat warnings as errors

mihirpathak97 · Oct 4, 2018 · 6f6aa72 · 6f6aa72
1 parent 6c7d88c
commit 6f6aa72
Showing 1 changed file with 15 additions and 14 deletions.
diff --git a/scripts/build.js b/scripts/build.js
@@ -150,20 +150,21 @@ function build(previousFileSizes) {
         }
         return reject(new Error(messages.errors.join('\n\n')));
       }
-      if (
-        process.env.CI &&
-        (typeof process.env.CI !== 'string' ||
-          process.env.CI.toLowerCase() !== 'false') &&
-        messages.warnings.length
-      ) {
-        console.log(
-          chalk.yellow(
-            '\nTreating warnings as errors because process.env.CI = true.\n' +
-              'Most CI servers set it automatically.\n'
-          )
-        );
-        return reject(new Error(messages.warnings.join('\n\n')));
-      }
+      // Do not treat warnings as error in CI build
+      // if (
+      //   process.env.CI &&
+      //   (typeof process.env.CI !== 'string' ||
+      //     process.env.CI.toLowerCase() !== 'false') &&
+      //   messages.warnings.length
+      // ) {
+      //   console.log(
+      //     chalk.yellow(
+      //       '\nTreating warnings as errors because process.env.CI = true.\n' +
+      //         'Most CI servers set it automatically.\n'
+      //     )
+      //   );
+      //   return reject(new Error(messages.warnings.join('\n\n')));
+      // }
 
       const resolveArgs = {
         stats,