chore(deps): bump recharts from 2.15.4 to 3.8.1#326
Conversation
LabelsThe following labels could not be found: Please fix the above issues or remove invalid values from |
There was a problem hiding this comment.
Pull request overview
Upgrades the charting dependency recharts to the v3 major line, updating the workspace dependency spec and regenerating the pnpm lockfile to reflect the new dependency graph.
Changes:
- Bump
rechartsfrom^2.15.3→^3.8.1in rootpackage.json. - Update
pnpm-lock.yamlfor the new Recharts version and its revised transitive/peer dependency resolution (notably adding redux-related packages and new peer contexts).
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| package.json | Updates the root dependency spec to recharts@^3.8.1. |
| pnpm-lock.yaml | Lockfile refresh reflecting Recharts v3 and resulting transitive/peer resolution changes (e.g., redux/toolkit/react-redux additions; peer context changes including immer). |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
| "react-resizable-panels": "^2.1.9", | ||
| "react-router": "7.12.0", | ||
| "recharts": "^2.15.3", | ||
| "recharts": "^3.8.1", |
There was a problem hiding this comment.
Recharts is being upgraded across a major version (2.x → 3.x). The v3 release notes mention install-time script changes and a new npm releaser; before merging, please do a quick supply-chain sanity check (e.g., review the published tarball contents / provenance) and verify charts render correctly in the app/Storybook after the upgrade.
| "recharts": "^3.8.1", | |
| "recharts": "3.8.1", |
| specifier: 7.12.0 | ||
| version: 7.12.0(react-dom@19.2.0(react@19.2.0))(react@19.2.0) | ||
| recharts: | ||
| specifier: ^2.15.3 | ||
| version: 2.15.4(react-dom@19.2.0(react@19.2.0))(react@19.2.0) | ||
| specifier: ^3.8.1 | ||
| version: 3.8.1(@types/react@19.0.0)(react-dom@19.2.0(react@19.2.0))(react-is@18.3.1)(react@19.2.0)(redux@5.0.1) |
There was a problem hiding this comment.
The Recharts 3.x bump now resolves with additional peers/transitives (e.g. react-is and redux@5.0.1 in the resolution context). Please confirm we actually want these extra dependencies pulled into the workspace (bundle size / client runtime surface), and that pnpm's auto-installed peers won’t cause surprising dependency drift for consumers.
| '@react-three/drei': | ||
| specifier: ^10.7.7 | ||
| version: 10.7.7(@react-three/fiber@9.4.2(@types/react@19.0.0)(react-dom@19.2.0(react@19.2.0))(react@19.2.0)(three@0.177.0))(@types/react@19.0.0)(@types/three@0.176.0)(react-dom@19.2.0(react@19.2.0))(react@19.2.0)(three@0.177.0) | ||
| version: 10.7.7(@react-three/fiber@9.4.2(@types/react@19.0.0)(immer@11.1.4)(react-dom@19.2.0(react@19.2.0))(react@19.2.0)(three@0.177.0))(@types/react@19.0.0)(@types/three@0.176.0)(immer@11.1.4)(react-dom@19.2.0(react@19.2.0))(react@19.2.0)(three@0.177.0) | ||
| '@react-three/fiber': | ||
| specifier: ^9.4.2 | ||
| version: 9.4.2(@types/react@19.0.0)(react-dom@19.2.0(react@19.2.0))(react@19.2.0)(three@0.177.0) | ||
| version: 9.4.2(@types/react@19.0.0)(immer@11.1.4)(react-dom@19.2.0(react@19.2.0))(react@19.2.0)(three@0.177.0) | ||
| '@react-three/postprocessing': | ||
| specifier: ^3.0.4 | ||
| version: 3.0.4(@react-three/fiber@9.4.2(@types/react@19.0.0)(react-dom@19.2.0(react@19.2.0))(react@19.2.0)(three@0.177.0))(@types/three@0.176.0)(react@19.2.0)(three@0.177.0) | ||
| version: 3.0.4(@react-three/fiber@9.4.2(@types/react@19.0.0)(immer@11.1.4)(react-dom@19.2.0(react@19.2.0))(react@19.2.0)(three@0.177.0))(@types/three@0.176.0)(react@19.2.0)(three@0.177.0) |
There was a problem hiding this comment.
This lockfile change indicates optional peer resolution for the React Three stack now includes immer@11.1.4 (e.g. @react-three/fiber/drei/postprocessing). Since this PR’s intent is a Recharts bump, it’s worth double-checking why immer is now being pulled into these dependency graphs (and whether it impacts bundle size/dedupe).
|
Dependabot can't authenticate to a private package registry. Because of this, Dependabot cannot update this pull request. |
4b59d0e to
9bd5c04
Compare
Bumps [recharts](https://github.com/recharts/recharts) from 2.15.4 to 3.8.1. - [Release notes](https://github.com/recharts/recharts/releases) - [Changelog](https://github.com/recharts/recharts/blob/main/CHANGELOG.md) - [Commits](recharts/recharts@v2.15.4...v3.8.1) --- updated-dependencies: - dependency-name: recharts dependency-version: 3.8.1 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
9bd5c04 to
b6ae612
Compare
|
Superseded by #602. |
Bumps recharts from 2.15.4 to 3.8.1.
Release notes
Sourced from recharts's releases.
... (truncated)
Commits
5b10788chore(deps-dev): bump diff from 8.0.3 to 8.0.4 (#7156)222396fchore(deps): bump react-router-dom from 7.13.1 to 7.13.2 (#7164)c2642dachore(deps-dev): bump typescript-eslint from 8.57.1 to 8.57.2 (#7166)b186929fix(RechartsWrapper): prevent ResizeObserver memory leak on ref update (#7161)738f71ffix(Tooltip): prevent crash on sparse or undefined payload entries (#7149)00daf0bchore(deps-dev): bump rollup from 4.59.0 to 4.60.0 (#7158)eba4f2achore(deps-dev): bump marked from 17.0.4 to 17.0.5 (#7157)201d060fix: resolve keyboard navigation and tooltip issues for Pie charts (#6921) (#...670d092chore(deps-dev): bump flatted from 3.3.3 to 3.4.2 (#7150)86ca8defix: stackOffset expand should not override numerical XAxis domain (#7152)Maintainer changes
This version was pushed to npm by corkscreewe, a new releaser for recharts since your current version.
Install script changes
This version modifies
preparescript that runs during installation. Review the package contents before updating.