some npm version updates - #428
Conversation
remove mui styles remove webpack
There was a problem hiding this comment.
🟡 Changes recommended
The resolutions field is ineffective under npm (npm ci), so the intended dependency pinning should be converted to npm overrides (or otherwise corrected) before merging.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This PR updates the library’s dependency set (and publishes a new major version), removing some previously pinned build tooling while bumping several React/MUI/testing/storybook packages.
Changes:
- Bump package version to
34.0.0. - Remove direct webpack-related pins/dependencies (e.g., webpack resolution and loaders) while updating Storybook and other dev tooling versions.
- Remove
@mui/stylesand update MUI/React/React Router versions in peer + dev dependencies.
File summaries
| File | Description |
|---|---|
| package.json | Version bump and dependency/peerDependency updates, including removal of @mui/styles and direct webpack-related deps/pins. |
Review details
- Files reviewed: 1/2 changed files
- Comments generated: 2
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| "resolutions": { | ||
| "**/@emotion/styled": "11.0.0", | ||
| "webpack": "5.88.2" | ||
| "**/@emotion/styled": "11.0.0" | ||
| }, |
There was a problem hiding this comment.
🟢 Approval recommended
The changes are confined to dependency/version declarations and appear internally consistent with the current build/test scripts and tooling setup.
Review details
- Files reviewed: 1/2 changed files
- Comments generated: 0 new
- Review effort level: Lite
remove mui styles
remove direct webpack deps