-
Notifications
You must be signed in to change notification settings - Fork 39
Update dep & fix vulnerabilities #569
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.
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
Conversation
Co-authored-by: Copilot <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates dependencies and fixes vulnerabilities by modernizing the project's tooling infrastructure. The changes migrate from ESLint 8 to 9 with new configuration patterns, update Jest from 27 to 30, upgrade TypeScript from 4.x to 5.x, and update various other dependencies including Babel, Webpack, and Storybook.
Key changes include:
- Migration to ESLint 9 with neostandard configuration, replacing outdated standard and node plugins
- Jest upgrade with environment configuration changes and test modifications
- Dependency version bumps across build tools and testing libraries
Reviewed Changes
Copilot reviewed 80 out of 85 changed files in this pull request and generated 2 comments.
Show a summary per file
File | Description |
---|---|
package.json | Updated dependency versions and build script commands |
eslint.config.mjs | New ESLint 9 configuration replacing legacy .eslintrc |
test files | Updated import paths from setup to debugger helper |
jest-environment-jsdom.js | Removed custom Jest environment file |
.babelrc | Updated Babel plugin names for compatibility |
.storybook/* | Updated Storybook configuration for v7 |
Various source files | Removed ESLint disable comments and minor formatting changes |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
updated prod dependencies to newest
updated eslint from version 8 to 9 -> migration of config file (see: https://eslint.org/docs/latest/use/configure/migration-guide) which includes:
** drop for standard because outdated, now using neostandard
** drop for node because discontinued, now using n.
typescript from 4.x to 5.x
jest from 27.x to 30.x with jsdom and jest-environment-test )skipped 2 tests because of too many Jest workers (needs fix sometime)
didn't manage to update nock because I get errors like 'ReferenceError: TransformStream is not defined'
didn't manage to update @testing-library/user-event" because tests give different results
updated babel
** babel proposal dep deprecated -> updated to transform
updated webpack, especially cli to 6.x
only updated Storybook from 6.x to 7.x (8 is not possible due to the usage of MDX stories, which is not supported anymore from 8). I created a separate issue for a future upgrade.
react is also dependent on storybook and only needed for it.
P.s. if build fails on publish -> not a concern