Prep for 1.0: upgrading all outdated dependencies

weaverryan · weaverryan · commit cfb557fc0dfc · 2021-01-27T15:05:43.000-05:00
diff --git a/.travis.yml b/.travis.yml
@@ -13,7 +13,6 @@ matrix:
       node_js: "10"
       env:
         - JOB_PART=test
-        - DISABLE_UNSTABLE_CHECKS=1
       install:
         - rm yarn.lock
         - node ./scripts/force-lowest-dependencies
@@ -23,7 +22,6 @@ matrix:
       node_js: "12"
       env:
         - JOB_PART=test
-        - DISABLE_UNSTABLE_CHECKS=1
       install:
         - rm yarn.lock
         - yarn
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,21 @@
 
  * [DEPENDENCY UPGRADE] Webpack was upgraded from version 4 to 5.
 
+* [DEPENDENCY UPGRADES] The following packages had major version upgrades:
+   * `css-loader` from 3 to 5
+   * `assets-webpack-plugin` from 5 to 7
+   * `mini-css-extract-plugin` from 0.4 to 1
+   * `style-loader` from 1 to 2
+   * `terser-webpack-plugin` from 1 to 5
+   * `webpack-cli` from 3 to 4
+   * `webpack-manifest-plugin` from 2 to 3
+   * `webpack-manifest-plugin` from 3 to 4-beta
+
+* [DEPENDENCY SUPPORT CHANGES] Encore has changed what versions it supports
+   of the following packages:
+   * `less` from 3 to 4 and `less-loader` from 6 to 7
+   * `fork-ts-checker-webpack-plugin` from 4 to 5 or 6
+
  * [BC BREAK] Image and font processing was changed from using `file-loader`
    (and  optionally `url-loader` via `configureUrlLoader()`) to Webpack 5's
    new [Asset Modules](https://webpack.js.org/guides/asset-modules/).
@@ -34,15 +49,6 @@
 
 * [BC BREAK] All previously-deprecated methods & options were removed.
 
-* [DEPENDENCY UPGRADES] The following packages had major version upgrades:
-   * `css-loader` from 3 to 5
-   * `mini-css-extract-plugin` from 0.4 to 1
-   * `style-loader` from 1 to 2
-   * `terser-webpack-plugin` from 1 to 4
-   * `webpack-cli` from 3 to 4
-   * `webpack-manifest-plugin` from 2 to 3
-   * `less` from 3 to 4 and `less-loader` from 6 to 7
-
 * [BEHAVIOR CHANGE] The `HashedModuleIdsPlugin` was previously used to
   help name "modules" when building for production. This has been removed
   and we now use Webpack's native `optimization.moduleIds` option, which
diff --git a/lib/config-generator.js b/lib/config-generator.js
@@ -379,7 +379,7 @@ class ConfigGenerator {
         if (this.webpackConfig.useEslintLoader) {
             rules.push(applyRuleConfigurationCallback('eslint', {
                 test: eslintLoaderUtil.getTest(this.webpackConfig),
-                loader: require.resolve('eslint-loader'),
+                loader: require.resolve('eslint-loader'), //eslint-disable-line node/no-unpublished-require
                 exclude: /node_modules/,
                 enforce: 'pre',
                 options: eslintLoaderUtil.getOptions(this.webpackConfig)
diff --git a/lib/loaders/css.js b/lib/loaders/css.js
@@ -54,7 +54,7 @@ module.exports = {
             };
 
             cssLoaders.push({
-                loader: require.resolve('postcss-loader'),
+                loader: require.resolve('postcss-loader'), //eslint-disable-line node/no-unpublished-require
                 options: applyOptionsCallback(webpackConfig.postCssLoaderOptionsCallback, postCssLoaderOptions)
             });
         }
diff --git a/lib/loaders/handlebars.js b/lib/loaders/handlebars.js
@@ -25,7 +25,7 @@ module.exports = {
 
         return [
             {
-                loader: require.resolve('handlebars-loader'),
+                loader: require.resolve('handlebars-loader'), //eslint-disable-line node/no-unpublished-require
                 options: applyOptionsCallback(webpackConfig.handlebarsConfigurationCallback, options)
             }
         ];
diff --git a/lib/loaders/less.js b/lib/loaders/less.js
@@ -30,7 +30,7 @@ module.exports = {
         return [
             ...cssLoader.getLoaders(webpackConfig, useCssModules),
             {
-                loader: require.resolve('less-loader'),
+                loader: require.resolve('less-loader'), //eslint-disable-line node/no-unpublished-require
                 options: applyOptionsCallback(webpackConfig.lessLoaderOptionsCallback, config)
             },
         ];
diff --git a/lib/loaders/sass.js b/lib/loaders/sass.js
@@ -49,7 +49,7 @@ module.exports = {
         });
 
         sassLoaders.push({
-            loader: require.resolve('sass-loader'),
+            loader: require.resolve('sass-loader'), //eslint-disable-line node/no-unpublished-require
             options: applyOptionsCallback(webpackConfig.sassLoaderOptionsCallback, config)
         });
 
diff --git a/lib/loaders/stylus.js b/lib/loaders/stylus.js
@@ -30,7 +30,7 @@ module.exports = {
         return [
             ...cssLoader.getLoaders(webpackConfig, useCssModules),
             {
-                loader: require.resolve('stylus-loader'),
+                loader: require.resolve('stylus-loader'), //eslint-disable-line node/no-unpublished-require
                 options: applyOptionsCallback(webpackConfig.stylusLoaderOptionsCallback, config)
             },
         ];
diff --git a/lib/loaders/typescript.js b/lib/loaders/typescript.js
@@ -51,7 +51,7 @@ module.exports = {
         let loaders = babelLoader.getLoaders(webpackConfig);
         return loaders.concat([
             {
-                loader: require.resolve('ts-loader'),
+                loader: require.resolve('ts-loader'), //eslint-disable-line node/no-unpublished-require
                 // @see https://github.com/TypeStrong/ts-loader/blob/master/README.md#available-options
                 options: config
             }
diff --git a/lib/loaders/vue.js b/lib/loaders/vue.js
@@ -27,7 +27,7 @@ module.exports = {
 
         return [
             {
-                loader: require.resolve('vue-loader'),
+                loader: require.resolve('vue-loader'), //eslint-disable-line node/no-unpublished-require
                 options: applyOptionsCallback(webpackConfig.vueLoaderOptionsCallback, options)
             }
         ];
diff --git a/lib/plugins/terser.js b/lib/plugins/terser.js
@@ -19,8 +19,6 @@ const applyOptionsCallback = require('../utils/apply-options-callback');
  */
 module.exports = function(webpackConfig) {
     const terserPluginOptions = {
-        sourceMap: webpackConfig.useSourceMaps,
-        cache: true,
         parallel: true,
     };
 
diff --git a/package.json b/package.json
@@ -28,30 +28,28 @@
     "@babel/core": "^7.7.0",
     "@babel/plugin-syntax-dynamic-import": "^7.0.0",
     "@babel/preset-env": "^7.10.0",
-    "assets-webpack-plugin": "^5.1.1",
+    "assets-webpack-plugin": "^7.0.0",
     "babel-loader": "^8.0.0",
     "chalk": "^4.0.0",
     "clean-webpack-plugin": "^3.0.0",
     "css-loader": "^5.0.1",
     "css-minimizer-webpack-plugin": "^1.1.5",
-    "fast-levenshtein": "^2.0.6",
+    "fast-levenshtein": "^3.0.0",
     "friendly-errors-webpack-plugin": "^2.0.0-beta.1",
     "loader-utils": "^2.0.0",
     "mini-css-extract-plugin": "^1.0.0",
     "pkg-up": "^3.1.0",
-    "pretty-error": "^2.1.1",
+    "pretty-error": "^3.0.3",
     "resolve-url-loader": "^3.1.2",
     "semver": "^7.3.2",
     "style-loader": "^2.0.0",
-    "terser-webpack-plugin": "^4.2.3",
+    "terser-webpack-plugin": "^5.1.1",
     "tmp": "^0.2.1",
     "webpack": "^5.12",
     "webpack-cli": "^4",
-    "webpack-dev-server": "^3.1.14",
+    "webpack-dev-server": "^4.0.0-beta.0",
     "webpack-manifest-plugin": "^3",
-    "webpack-sources": "^1.3.0",
-    "webpack-virtual-modules": "^0.2.2",
-    "yargs-parser": "^18.1.3"
+    "yargs-parser": "^20.2.4"
   },
   "devDependencies": {
     "@babel/plugin-proposal-class-properties": "^7.0.0",
@@ -63,36 +61,37 @@
     "@vue/babel-helper-vue-jsx-merge-props": "^1.0.0",
     "@vue/babel-preset-jsx": "^1.0.0",
     "@vue/compiler-sfc": "^3.0.2",
-    "autoprefixer": "^8.5.0",
+    "autoprefixer": "^10.2.0",
     "babel-eslint": "^10.0.1",
     "chai": "^4.2.0",
     "chai-fs": "^2.0.0",
     "chai-subset": "^1.6.0",
     "core-js": "^3.0.0",
     "eslint": "^6.0.0 || ^7.0.0",
     "eslint-loader": "^4.0.0",
-    "eslint-plugin-header": "^1.0.0",
+    "eslint-plugin-header": "^3.0.0",
     "eslint-plugin-import": "^2.8.0",
-    "eslint-plugin-node": "^8.0.1",
+    "eslint-plugin-node": "^11.1.0",
     "file-loader": "^6.0.0",
-    "fork-ts-checker-webpack-plugin": "^4.0.0",
+    "fork-ts-checker-webpack-plugin": "^5.0.0 || ^6.0.0",
     "fs-extra": "^9.0.0",
     "handlebars": "^4.0.11",
     "handlebars-loader": "^1.7.0",
     "http-server": "^0.12.3",
     "less": "^4.0.0",
     "less-loader": "^7.0.0",
-    "mocha": "^7.1.2",
+    "mocha": "^8.2.1",
+    "postcss": "^8.1.0",
     "postcss-loader": "^4.0.0",
-    "preact": "^8.2.1",
+    "preact": "^8.2.1 || ^10.0.0",
     "preact-compat": "^3.17.0",
     "sass": "^1.17.0",
     "sass-loader": "^9.0.1 || ^10.0.0",
     "sinon": "^9.0.2",
     "stimulus": "^2.0.0",
     "strip-ansi": "^6.0.0",
     "stylus": "^0.54.5",
-    "stylus-loader": "^3.0.2",
+    "stylus-loader": "^3.0.2 || ^4.0.0",
     "ts-loader": "^8.0.1",
     "typescript": ">=3.6.3",
     "vue": "^3.0.2",
diff --git a/test/functional.js b/test/functional.js
diff --git a/test/package-helper.js b/test/package-helper.js
diff --git a/test/plugins/forked-ts-types.js b/test/plugins/forked-ts-types.js
diff --git a/test/plugins/terser.js b/test/plugins/terser.js
diff --git a/yarn.lock b/yarn.lock

Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ module.exports = {`
`54`	`54`	`};`
`55`	`55`
`56`	`56`	`cssLoaders.push({`
`57`		`- loader: require.resolve('postcss-loader'),`
	`57`	`+ loader: require.resolve('postcss-loader'), //eslint-disable-line node/no-unpublished-require`
`58`	`58`	`options: applyOptionsCallback(webpackConfig.postCssLoaderOptionsCallback, postCssLoaderOptions)`
`59`	`59`	`});`
`60`	`60`	`}`
Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@ module.exports = {`
`25`	`25`
`26`	`26`	`return [`
`27`	`27`	`{`
`28`		`- loader: require.resolve('handlebars-loader'),`
	`28`	`+ loader: require.resolve('handlebars-loader'), //eslint-disable-line node/no-unpublished-require`
`29`	`29`	`options: applyOptionsCallback(webpackConfig.handlebarsConfigurationCallback, options)`
`30`	`30`	`}`
`31`	`31`	`];`
Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,7 @@ module.exports = {`
`30`	`30`	`return [`
`31`	`31`	`...cssLoader.getLoaders(webpackConfig, useCssModules),`
`32`	`32`	`{`
`33`		`- loader: require.resolve('less-loader'),`
	`33`	`+ loader: require.resolve('less-loader'), //eslint-disable-line node/no-unpublished-require`
`34`	`34`	`options: applyOptionsCallback(webpackConfig.lessLoaderOptionsCallback, config)`
`35`	`35`	`},`
`36`	`36`	`];`
Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ module.exports = {`
`51`	`51`	`let loaders = babelLoader.getLoaders(webpackConfig);`
`52`	`52`	`return loaders.concat([`
`53`	`53`	`{`
`54`		`- loader: require.resolve('ts-loader'),`
	`54`	`+ loader: require.resolve('ts-loader'), //eslint-disable-line node/no-unpublished-require`
`55`	`55`	`// @see https://github.com/TypeStrong/ts-loader/blob/master/README.md#available-options`
`56`	`56`	`options: config`
`57`	`57`	`}`
Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ module.exports = {`
`27`	`27`
`28`	`28`	`return [`
`29`	`29`	`{`
`30`		`- loader: require.resolve('vue-loader'),`
	`30`	`+ loader: require.resolve('vue-loader'), //eslint-disable-line node/no-unpublished-require`
`31`	`31`	`options: applyOptionsCallback(webpackConfig.vueLoaderOptionsCallback, options)`
`32`	`32`	`}`
`33`	`33`	`];`