Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: bump all webpack related dependencies to latest #30684

Merged
merged 7 commits into from
Mar 6, 2024
Merged

chore: bump all webpack related dependencies to latest #30684

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
01d2aea
chore: bump webpack n friends
Hotell Feb 27, 2024
c961ef0
chore: bump all webpack related packages to latest
Hotell Mar 1, 2024
88fb079
chore: bump express for further dep dedupe
Hotell Mar 1, 2024
7fb0c7b
fix: resolve type errors in webpack configs caused by more strict typ…
Hotell Mar 1, 2024
05f6c9f
fix(fluentui/docs): resolve type issues after webpack bump
Hotell Mar 1, 2024
26e6283
fix(scripts-gulp): resolve type/logic issues after webpack middleware…
Hotell Mar 1, 2024
d6cc9dd
chore: revert webpack-dev-server v5 to latest v4 in order to make cyp…
Hotell Mar 1, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .storybook/main.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ module.exports = /** @type {Omit<StorybookConfig,'typescript'|'babel'>} */ ({

if ((process.env.CI || process.env.TF_BUILD || process.env.LAGE_PACKAGE_NAME) && config.plugins) {
// Disable ProgressPlugin in PR/CI builds to reduce log verbosity (warnings and errors are still logged)
config.plugins = config.plugins.filter(({ constructor }) => constructor.name !== 'ProgressPlugin');
config.plugins = config.plugins.filter(value => value && value.constructor.name !== 'ProgressPlugin');
Hotell marked this conversation as resolved.
Show resolved Hide resolved
}

return config;
Expand Down
44 changes: 21 additions & 23 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -134,13 +134,13 @@
"@types/babel__traverse": "7.20.3",
"@types/chrome-remote-interface": "0.30.0",
"@types/circular-dependency-plugin": "5.0.5",
"@types/copy-webpack-plugin": "6.4.0",
"@types/copy-webpack-plugin": "10.1.0",
"@types/dedent": "0.7.0",
"@types/doctrine": "0.0.5",
"@types/ejs": "3.1.2",
"@types/enzyme": "3.10.7",
"@types/eslint": "7.2.13",
"@types/express": "4.17.15",
"@types/express": "4.17.21",
"@types/fs-extra": "8.0.1",
"@types/glob": "7.1.1",
"@types/graphviz": "0.0.34",
Expand Down Expand Up @@ -172,10 +172,10 @@
"@types/semver": "^6.2.0",
"@types/tmp": "0.2.0",
"@types/vinyl": "2.0.7",
"@types/webpack-bundle-analyzer": "4.4.3",
"@types/webpack-dev-middleware": "4.1.0",
"@types/webpack-env": "1.16.0",
"@types/webpack-hot-middleware": "2.25.6",
"@types/webpack-bundle-analyzer": "4.7.0",
"@types/webpack-dev-middleware": "5.3.0",
"@types/webpack-env": "1.18.4",
"@types/webpack-hot-middleware": "2.25.9",
"@types/yargs": "13.0.11",
"@types/yargs-unparser": "2.0.1",
"@typescript-eslint/eslint-plugin": "4.22.0",
Expand All @@ -200,7 +200,7 @@
"clean-css": "4.2.3",
"clean-webpack-plugin": "4.0.0",
"connect-history-api-fallback": "1.6.0",
"copy-webpack-plugin": "8.1.0",
"copy-webpack-plugin": "12.0.2",
"cross-env": "^5.1.4",
"css-loader": "5.0.1",
"cypress": "13.6.4",
Expand Down Expand Up @@ -230,12 +230,12 @@
"eslint-plugin-jsx-a11y": "6.4.1",
"eslint-plugin-react": "7.26.0",
"eslint-plugin-react-hooks": "4.2.0",
"express": "4.17.3",
"express": "4.18.3",
"extract-comments": "1.1.0",
"file-loader": "6.2.0",
"find-free-port": "2.0.0",
"flamegrill": "0.2.0",
"fork-ts-checker-webpack-plugin": "6.1.0",
"fork-ts-checker-webpack-plugin": "9.0.2",
"fs-extra": "8.1.0",
"geckodriver": "3.0.2",
"glob": "7.2.0",
Expand All @@ -246,7 +246,7 @@
"gulp-remember": "1.0.1",
"gulp-sourcemaps": "2.6.5",
"gulp-util": "3.0.8",
"html-webpack-plugin": "5.1.0",
"html-webpack-plugin": "5.6.0",
"ignore-not-found-export-webpack-plugin": "1.0.2",
"imports-loader": "1.2.0",
"jest": "29.7.0",
Expand All @@ -262,7 +262,7 @@
"jsonc-eslint-parser": "2.3.0",
"just-scripts": "1.8.2",
"lage": "1.8.8",
"license-webpack-plugin": "2.3.10",
"license-webpack-plugin": "4.0.2",
"lint-staged": "10.2.10",
"loader-utils": "2.0.4",
"lodash": "4.17.21",
Expand All @@ -272,10 +272,8 @@
"monosize": "0.3.0",
"monosize-storage-azure": "0.0.8",
"node-plop": "0.25.0",
"node-polyfill-webpack-plugin": "1.0.2",
"nx": "17.3.2",
"p-queue": "6.6.2",
"parallel-webpack": "2.6.0",
"parse-diff": "0.7.1",
"path-browserify": "1.0.1",
"plop": "2.6.0",
Expand Down Expand Up @@ -324,26 +322,26 @@
"swc-plugin-de-indent-template-literal": "1.0.0",
"syncpack": "10.6.1",
"tachometer": "0.7.0",
"terser": "5.27.2",
"terser": "5.28.1",
"terser-webpack-plugin": "5.3.10",
"through2": "4.0.2",
"tmp": "0.2.1",
"ts-jest": "29.1.1",
"ts-loader": "9.3.1",
"ts-node": "10.9.1",
"tsconfig-paths": "4.1.0",
"tsconfig-paths-webpack-plugin": "4.0.0",
"tsconfig-paths": "4.2.0",
"tsconfig-paths-webpack-plugin": "4.1.0",
"tslib": "2.5.0",
"typescript": "4.7.4",
"vinyl": "2.2.0",
"vrscreenshotdiff": "0.0.17",
"webpack": "5.83.1",
"webpack-bundle-analyzer": "4.4.2",
"webpack-cli": "4.10.0",
"webpack-dev-middleware": "4.2.0",
"webpack-dev-server": "4.7.4",
"webpack-hot-middleware": "2.25.1",
"webpack-merge": "5.7.3",
"webpack": "5.90.3",
"webpack-bundle-analyzer": "4.10.1",
"webpack-cli": "5.1.4",
"webpack-dev-middleware": "7.0.0",
"webpack-dev-server": "4.15.1",
"webpack-hot-middleware": "2.26.1",
"webpack-merge": "5.10.0",
"workspace-tools": "0.29.1",
"yargs": "13.3.2",
"yargs-parser": "13.1.2",
Expand Down
2 changes: 1 addition & 1 deletion packages/fluentui/docs/src/contexts/exampleSourcesContext.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ if (__DEV__) {
if (module.hot) {
// We need this to catch cases unhandled by RHL
// https://github.com/webpack/webpack/issues/834#issuecomment-76590576
module.hot.accept(exampleSourcesContext.id, () => {
module.hot.accept(exampleSourcesContext.id as string, () => {
exampleSourcesContext = require.context('../exampleSources/', true, /.source.json$/);
});
}
Expand Down
2 changes: 1 addition & 1 deletion packages/fluentui/docs/src/contexts/examplesContext.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ if (__DEV__) {
if (module.hot) {
// We need this to catch cases unhandled by RHL
// https://github.com/webpack/webpack/issues/834#issuecomment-76590576
module.hot.accept(examplesContext.id, () => {
module.hot.accept(examplesContext.id as string, () => {
examplesContext = require.context('../examples/', true, /(\w+Example(\w|\.)*|\w+.perf|\w+.bsize)\.tsx$/);
});
}
Expand Down
2 changes: 1 addition & 1 deletion scripts/gulp/src/tasks/docs.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -193,7 +193,7 @@ task('serve:docs:hot', async () => {
WebpackDevMiddleware(compiler, {
publicPath: webpackConfig.output.publicPath,
stats: 'errors-warnings',
} as WebpackDevMiddleware.Options),
} as WebpackDevMiddleware.Options<any, any>),
Hotell marked this conversation as resolved.
Show resolved Hide resolved
);

if (process.env.NODE_ENV !== 'production') {
Expand Down
12 changes: 8 additions & 4 deletions scripts/gulp/src/tasks/test-dependencies/utils.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,12 +45,16 @@ export const prepareWebpackConfig = (options: WebpackOptions) => {
},
renderLicenses: modules => {
modules.forEach(module => {
const packageName = module.packageJson.name;
const packageVersion = module.packageJson.version;
if (module.packageJson) {
Hotell marked this conversation as resolved.
Show resolved Hide resolved
const packageName = module.packageJson.name;
const packageVersion = module.packageJson.version;

onDependencyPackage(packageName, packageVersion);
onDependencyPackage(packageName, packageVersion);
} else {
console.error('package.json not found');
}

return modules[0].packageJson.name;
return modules[0].packageJson?.name;
});

return '';
Expand Down
4 changes: 2 additions & 2 deletions scripts/webpack/src/storybook-webpack.config.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,9 @@ const createStorybookWebpackConfig = config => {

config.plugins = [...(config.plugins || []), new IgnoreNotFoundExportWebpackPlugin({ include: [/\.tsx?$/] })];

// Disable ProgressPlugin which logs verbose webpack build progress. Warnings and Errors are still logged.
if (process.env.TF_BUILD || process.env.LAGE_PACKAGE_NAME) {
config.plugins = config.plugins.filter(({ constructor }) => constructor.name !== 'ProgressPlugin');
// Disable ProgressPlugin in PR/CI builds to reduce log verbosity (warnings and errors are still logged)
config.plugins = config.plugins.filter(value => value && value.constructor.name !== 'ProgressPlugin');
}

config.optimization = { ...config.optimization, minimize: false };
Expand Down
Loading
Loading