Skip to content

[Data Weaver] Dependency updates - #431

Open
nick-nlb wants to merge 3 commits into
datacommonsorg:mainfrom
nick-nlb:dependency-updates
Open

[Data Weaver] Dependency updates#431
nick-nlb wants to merge 3 commits into
datacommonsorg:mainfrom
nick-nlb:dependency-updates

Conversation

@nick-nlb

@nick-nlb nick-nlb commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Description

This PR updates dependencies for Data Weaver.

Notes

The canary version of Next.js upgrades the two dependencies that this PR overrides (in order to push the versions out of vulnerability). Once this canary becomes stable we can upgrade and remove the overrides.

…te that canary pins these updated versions natively, meaning. that once canary becomes stable Next.js, we can remove these overrides.
@nick-nlb
nick-nlb requested a review from juliawu August 3, 2026 14:40

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates several dependencies across the workspace and introduces pnpm overrides for sharp and postcss. The review feedback identifies two critical issues: the invalid workspace:^ syntax used for @package/tokens which should be updated to workspace:*, and a non-existent version 8.5.18 specified for postcss in the pnpm overrides which will cause installation to fail.

Comment thread dataweaver/apps/web/package.json
Comment thread dataweaver/package.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants