-
Notifications
You must be signed in to change notification settings - Fork 6
fix(deps): update dependency react-svgmt to v1.2.0 #268
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
base: main
Are you sure you want to change the base?
Conversation
|
📝 WalkthroughWalkthroughThe pull request updates a single dependency in package.json: 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
|
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.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
package.json (1)
2544-2550: Incompatible React peer dependency.react-svgmt v1.2.0 requires React 17 as a peer dependency, but this project uses React 16.5.2. Update React or pin react-svgmt to a compatible version.
📜 Review details
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Disabled knowledge base sources:
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
package.json
5ae9c14 to
8cff953
Compare
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.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In @package.json:
- Line 2550: The package.json currently pins "react-svgmt": "1.2.0" which
requires React ^17.0.0 while the project dependencies still pin React/ReactDOM
at ^16.5.2; update the React and ReactDOM entries in package.json to a
v17-compatible range (e.g., ^17.0.0) or downgrade react-svgmt to a version
compatible with React 16, then run install and test; specifically modify the
"react" and "react-dom" dependency lines in package.json to match React 17 to
resolve the peer dependency mismatch and ensure package-lock/yarn.lock is
regenerated.
📜 Review details
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Disabled knowledge base sources:
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
package.json
02ccdb1 to
caf8930
Compare
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.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In @package.json:
- Line 2650: package.json currently pins "react-svgmt": "1.2.0" which requires
React v17 while the project lists "react": "16.5.2"; either replace
"react-svgmt": "1.2.0" with a version compatible with React 16 (find the latest
16-compatible react-svgmt and update that dependency) or upgrade the project's
React dependency to ^17.0.0 and update any breaking changes; update package.json
accordingly and run npm/yarn install and tests to verify.
📜 Review details
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Disabled knowledge base sources:
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
package.json
350d3e8 to
8ece1f0
Compare
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.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
package.json (1)
2644-2650: react-svgmt 1.2.0 requires React v17.0.0 but project uses v16.5.2.Upgrading react-svgmt to 1.2.0 will cause unmet peer dependency warnings/errors. React and react-dom must be upgraded to v17.0.0 or the react-svgmt update should be reverted.
📜 Review details
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Disabled knowledge base sources:
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
package.json
8ece1f0 to
606bb32
Compare
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.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In @package.json:
- Line 2650: The package.json update added "react-svgmt": "1.2.0" which requires
React ^17 but this project uses React 16.5.2; either revert the dependency to a
16-compatible release (replace "react-svgmt": "1.2.0" with the last
v16-compatible tag, e.g., "react-svgmt": "1.1.0") or instead upgrade the
project's React and ReactDOM to ^17.x (update package.json entries for "react"
and "react-dom" and run tests); ensure whichever approach you choose resolves
peer dependency warnings and runs the app/tests successfully.
📜 Review details
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Disabled knowledge base sources:
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
package.json
539aa39 to
5b37fc1
Compare
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.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In @package.json:
- Line 2650: The project is using React ^16.5.2 while [email protected] requires
React ^17; update the peer dependency by bumping react and react-dom entries in
package.json to a compatible v17 version (e.g., ^17.0.0), then update lockfile
(package-lock.json or yarn.lock) by running the package manager install to
reconcile versions and run tests/build to verify compatibility; ensure
references in any bundler configs or import checks remain valid after the
upgrade.
📜 Review details
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Disabled knowledge base sources:
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
package.json
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.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In @package.json:
- Line 2650: The project currently uses react v16.5.2 while the added dependency
"react-svgmt": "1.2.0" requires React v17; fix by either updating package.json
to bump "react" and "react-dom" to a compatible v17.x version (and run
tests/build to address any breaking changes), or revert the new package to
"react-svgmt": "1.1.11" in package.json so it remains compatible with React
v16.5.2; update the same dependency entry ("react-svgmt") and the
"react"/"react-dom" entries accordingly and run install to validate.
📜 Review details
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Disabled knowledge base sources:
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
package.json
5b37fc1 to
c9ede73
Compare
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.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
package.json (1)
2644-2650: React version mismatch: react-svgmt v1.2.0 requires React v17.0.0.react-svgmt v1.2.0 declares peer dependencies on React v17.0.0 and ReactDOM v17.0.0, but this project uses v16.5.2. This will cause peer dependency conflicts and potential runtime errors.
Upgrade React and ReactDOM to v17.0.0, or revert react-svgmt to v1.1.11.
📜 Review details
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Disabled knowledge base sources:
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
package.json
c9ede73 to
99259b3
Compare
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.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In @package.json:
- Line 2685: The dependency "react-svgmt": "1.2.0" requires React 17+, but the
project uses React 16.5.2; update package.json to either (A) bump "react" and
"react-dom" to a ^17.0.0 version and then run your package manager install and
run tests, or (B) downgrade "react-svgmt" to a version compatible with React 16
(replace "react-svgmt": "1.2.0" with the older compatible version); ensure you
adjust any breaking API usage in code referencing React/ReactDOM or react-svgmt
and confirm CI/tests pass before merging.
📜 Review details
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Disabled knowledge base sources:
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
package.json
| "react-redux": "^7.1.1", | ||
| "react-svg-pan-zoom": "3.9.0", | ||
| "react-svgmt": "1.1.11", | ||
| "react-svgmt": "1.2.0", |
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.
🧩 Analysis chain
🏁 Script executed:
npm info [email protected] peerDependenciesRepository: deepnote/vscode-deepnote
Length of output: 137
🏁 Script executed:
rg -n '"react":|"react-dom":' package.json | head -20Repository: deepnote/vscode-deepnote
Length of output: 137
Upgrade React and ReactDOM before merging.
react-svgmt 1.2.0 requires React 17.0.0+, but this project uses React 16.5.2. Upgrade both react and react-dom to ^17.0.0, or downgrade react-svgmt to a compatible version.
🤖 Prompt for AI Agents
In @package.json at line 2685, The dependency "react-svgmt": "1.2.0" requires
React 17+, but the project uses React 16.5.2; update package.json to either (A)
bump "react" and "react-dom" to a ^17.0.0 version and then run your package
manager install and run tests, or (B) downgrade "react-svgmt" to a version
compatible with React 16 (replace "react-svgmt": "1.2.0" with the older
compatible version); ensure you adjust any breaking API usage in code
referencing React/ReactDOM or react-svgmt and confirm CI/tests pass before
merging.
This PR contains the following updates:
1.1.11→1.2.0Release Notes
hugozap/react-svgmt (react-svgmt)
v1.2.0Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.