-
Notifications
You must be signed in to change notification settings - Fork 574
Version Packages #7826
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
Version Packages #7826
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,12 @@ | ||
# @thirdweb-dev/nebula | ||
|
||
## 0.2.32 | ||
|
||
### Patch Changes | ||
|
||
- Updated dependencies [[`0d7e5e5`](https://github.com/thirdweb-dev/js/commit/0d7e5e563d142d178005469e31ea3423e6daacf1), [`a77e98a`](https://github.com/thirdweb-dev/js/commit/a77e98a48a2f10d8b465e2ef1ba0f28a6df43f3f), [`66c7df9`](https://github.com/thirdweb-dev/js/commit/66c7df98e42dc7b5168b817e3f9a179349479d62), [`90f36bc`](https://github.com/thirdweb-dev/js/commit/90f36bc414a8f29b9794d3d6a15e2b3e145eca7c)]: | ||
- [email protected] | ||
|
||
## 0.2.31 | ||
|
||
### Patch Changes | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -409,5 +409,5 @@ | |
} | ||
}, | ||
"typings": "./dist/types/exports/thirdweb.d.ts", | ||
"version": "5.105.31" | ||
"version": "5.105.32" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 💡 Verification agent 🧩 Analysis chainPatch release version updated Set to 5.105.32, matching the new changelog entry. LGTM. To ensure the deprecation of Bridge.routes sortBy is fully reflected, you can scan for remaining references and update JSDoc/types as needed: 🏁 Script executed: #!/bin/bash
# Search for Bridge.routes and any 'sortBy' usage across the repo
rg -n --no-ignore -S $'Bridge\\.routes\\(|sortBy' | sed 's/^/~/'
# Optionally, list TS definitions mentioning sortBy
rg -n --no-ignore -S $'sortBy' --glob '*.ts' --glob '*.d.ts' Length of output: 12766 Ensure complete deprecation of the Bridge.routes Although the version was bumped to 5.105.32, references to • packages/thirdweb/src/bridge/Routes.ts • packages/thirdweb/src/react/core/hooks/useBridgeRoutes.ts (+ its test) • apps/dashboard/src/(app)/(dashboard)/(bridge)/utils.ts 🤖 Prompt for AI Agents
|
||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,7 @@ | ||
# @thirdweb-dev/wagmi-adapter | ||
|
||
## 0.2.129 | ||
|
||
## 0.2.128 | ||
|
||
## 0.2.127 | ||
|
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.
💡 Verification agent
🧩 Analysis chain
Add deprecation guidance and timeline for
Bridge.routes({ sortBy })
.Deprecation notices are more useful with next-steps and a rough removal target. Suggest clarifying default behavior and when removal may occur. Also add a terminal period for consistency.
Proposed tweak:
Optional verification script to ensure code is annotated with a deprecation marker:
🏁 Script executed:
Length of output: 60883
Update CHANGELOG.md with deprecation guidance and timeline
The deprecation notice for
sortBy
inBridge.routes
should include next steps, a removal target, default behavior, and end with a period for consistency.• File: packages/thirdweb/CHANGELOG.md
Line: 7
📝 Committable suggestion
🤖 Prompt for AI Agents