Repository navigation
chore(deps): update dependencies for main branch - #12703
rebeccaalpert wants to merge 2 commits into
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters: 4
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including
|
| Layer / File(s) | Summary |
|---|---|
Pin lodash resolution package.json |
The root Yarn resolution pins lodash to 4.18.1. |
React docs ECharts dependency
| Layer / File(s) | Summary |
|---|---|
Update ECharts dependency range packages/react-docs/package.json |
The echarts range changes from `^5.6.0 |
Priority:
Estimated code review effort: 2 (Simple) | ~8 minutes
Change: Other
Possibly related PRs
- patternfly/patternfly-react#12697: It updates
packages/react-chartsto requirelodash^4.18.1, which aligns with the root resolution change.
Suggested reviewers: kmcfaul, wise-king-sullyman
Merge Risk: 
Minimal · previous review 5bb43
-
The dependency updates are ready to merge with no identified actionable risk.
Pre-merge checks: 
4
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
|
Preview: https://pf-react-pr-12703.surge.sh A11y report: https://pf-react-pr-12703-a11y.surge.sh |
Victory is still using an old version of lodash. I can put up a PR with them, but this should help us in the meantime.
See FormidableLabs/victory#3128.
What: Closes #
Additional issues:
Summary by CodeRabbit