Skip to content

dependencies#137

Merged
bourgeoa merged 2 commits intomainfrom
3.0.0-updates
Jan 25, 2026
Merged

dependencies#137
bourgeoa merged 2 commits intomainfrom
3.0.0-updates

Conversation

@bourgeoa
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings January 25, 2026 09:57
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates npm metadata and refreshes dependency versions to newer releases.

Changes:

  • Update bugs.url and homepage links to the SolidOS/issue-pane GitHub org/repo.
  • Bump runtime, peer, and dev dependency version ranges (Babel, ESLint, rdflib/solid-* packages, webpack-dev-server).
  • Regenerate package-lock.json to reflect updated transitive dependencies (including @noble/*, uuid, etc.).

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
package.json Updates repository metadata URLs and bumps direct/peer/dev dependency ranges.
package-lock.json Updates the resolved dependency tree and versions to match the new ranges.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 49 to +52
"peerDependencies": {
"rdflib": "^2.3.0",
"solid-logic": "^4.0.0",
"solid-ui": "^3.0.0"
"rdflib": "^2.3.5",
"solid-logic": "^4.0.2",
"solid-ui": "^3.0.3"
Copy link

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

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

Updating solid-ui/solid-logic peer versions appears to pull in @noble/* v2 (via solid-ui 3.0.3), which in package-lock now declares an engines requirement of Node ">= 20.19.0". This conflicts with the repo’s .nvmrc (v18.19.0) and may break local installs/builds for contributors on Node 18. Consider either (a) bumping the project’s documented/declared Node version (e.g., add package.json engines + update .nvmrc) or (b) pinning solid-ui (or overriding @noble/* back to v1.x) to keep Node 18 compatibility.

Copilot uses AI. Check for mistakes.
@bourgeoa bourgeoa merged commit 9c24db9 into main Jan 25, 2026
5 checks passed
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.

1 participant