-
Notifications
You must be signed in to change notification settings - Fork 41
302updates #670
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
302updates #670
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,6 +1,6 @@ | ||||||||||
| { | ||||||||||
| "name": "solid-ui", | ||||||||||
| "version": "3.0.2", | ||||||||||
| "version": "3.0.3", | ||||||||||
| "description": "UI library for Solid applications", | ||||||||||
| "main": "dist/solid-ui.js", | ||||||||||
| "types": "dist/index.d.ts", | ||||||||||
|
|
@@ -74,8 +74,8 @@ | |||||||||
| "uuid": "^13.0.0" | ||||||||||
| }, | ||||||||||
| "peerDependencies": { | ||||||||||
| "rdflib": "^2.3.0", | ||||||||||
| "solid-logic": "^4.0.1" | ||||||||||
| "rdflib": "^2.3.5", | ||||||||||
| "solid-logic": "^4.0.2" | ||||||||||
|
Comment on lines
+77
to
+78
|
||||||||||
| "rdflib": "^2.3.5", | |
| "solid-logic": "^4.0.2" | |
| "rdflib": "^2.3.0", | |
| "solid-logic": "^4.0.1" |
Copilot
AI
Jan 24, 2026
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.
rdflib and solid-logic are imported throughout src/ and test/ (e.g., many files import from these packages). Removing them from devDependencies can make npm test/npm run build fail in a fresh checkout unless the package manager installs root peerDependencies into the workspace. Consider adding them back to devDependencies (keeping them in peerDependencies for consumers) so contributors/CI can reliably install and run builds/tests.
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.
The package version is bumped as a patch release (
3.0.2→3.0.3), but tightening peer dependency minimums can be breaking for downstream apps. If this peer range narrowing is intentional, consider whether the package version should be a minor/major bump per semver (or document the breaking requirement in release notes).