test: do not treat warnings as errors on ci

Author: sergeysova

scripts/build.js

@@ -153,20 +153,20 @@ 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 !== 0
-      ) {
-        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")))
-      }
+      // if (
+      //   process.env.CI &&
+      //   (typeof process.env.CI !== "string" ||
+      //     process.env.CI.toLowerCase() !== "false") &&
+      //   messages.warnings.length !== 0
+      // ) {
+      //   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,