dependencies: bump Webpack to 5

client/scripts/build.js

@@ -9,14 +9,10 @@ process.on('unhandledRejection', (err) => {
   throw err;
 });
 
-// Ensure environment variables are read.
-require('../config/env');
-
 const chalk = require('chalk');
 const fs = require('fs-extra');
 const webpack = require('webpack');
 const checkRequiredFiles = require('react-dev-utils/checkRequiredFiles');
-const formatWebpackMessages = require('react-dev-utils/formatWebpackMessages');
 const FileSizeReporter = require('react-dev-utils/FileSizeReporter');
 const paths = require('../../shared/config/paths');
 const config = require('../config/webpack.config.prod');
@@ -49,26 +45,10 @@ const build = (previousFileSizes) => {
       if (err) {
         return reject(err);
       }
-      const messages = formatWebpackMessages(stats.toJson({}, true));
-      if (messages.errors.length) {
-        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')));
-      }
+
       return resolve({
         stats,
         previousFileSizes,
-        warnings: messages.warnings,
       });
     });
   });
@@ -87,14 +67,13 @@ measureFileSizesBeforeBuild(paths.appBuild)
     return build(previousFileSizes);
   })
   .then(
-    ({stats, previousFileSizes, warnings}) => {
-      if (warnings.length) {
+    ({stats, previousFileSizes}) => {
+      if (stats.compilation.warnings.length !== 0) {
         console.log(chalk.yellow('Compiled with warnings.\n'));
-        console.log(warnings.join('\n\n'));
-        console.log(`\nSearch for the ${chalk.underline(chalk.yellow('keywords'))} to learn more about each warning.`);
-        console.log(`To ignore, add ${chalk.cyan('// eslint-disable-next-line')} to the line before.\n`);
-      } else {
-        console.log(chalk.green('Compiled successfully.\n'));
+
+        stats.compilation.warnings.forEach((warning) => {
+          console.warn(warning);
+        });
       }
 
       console.log('File sizes after gzip:\n');

client/scripts/start.js

@@ -9,15 +9,12 @@ process.on('unhandledRejection', (err) => {
   throw err;
 });
 
-// Ensure environment variables are read.
-require('../config/env');
-
 const chalk = require('chalk');
 const webpack = require('webpack');
 const WebpackDevServer = require('webpack-dev-server');
 const clearConsole = require('react-dev-utils/clearConsole');
 const checkRequiredFiles = require('react-dev-utils/checkRequiredFiles');
-const {choosePort, createCompiler, prepareProxy, prepareUrls} = require('react-dev-utils/WebpackDevServerUtils');
+const {choosePort, prepareProxy, prepareUrls} = require('react-dev-utils/WebpackDevServerUtils');
 const openBrowser = require('react-dev-utils/openBrowser');
 const paths = require('../../shared/config/paths');
 const config = require('../config/webpack.config.dev');
@@ -45,11 +42,9 @@ choosePort(HOST, DEFAULT_PORT)
     }
 
     const protocol = process.env.DEV_SERVER_HTTPS === 'true' ? 'https' : 'http';
-    // eslint-disable-next-line import/no-dynamic-require
-    const appName = require(paths.appPackageJson).name;
     const urls = prepareUrls(protocol, HOST, port);
     // Create a webpack compiler that is configured with custom messages.
-    const compiler = createCompiler({webpack, config, appName, urls});
+    const compiler = webpack(config);
     // Load proxy config
     const proxyConfig = prepareProxy(userConfig.floodServerProxy, paths.appPublic);
     // Serve webpack assets generated by the compiler over a web sever.