-
Notifications
You must be signed in to change notification settings - Fork 2.7k
chore: add changeset for v3.26.0 #7428
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
Conversation
| - Fix: Make auto approve toggle trigger stay (#3909 by @kyle-apex, PR by @elianiva) | ||
| - Fix: Preserve user input when selecting follow-up choices (#7316 by @teihome, PR by @daniel-lxs) | ||
| - Fix: Resolve newTaskRequireTodos setting not working correctly (thanks @hannesrudolph!) | ||
| - Fix: Requesty model listing (#7377 by @dtrugman, PR by @dtrugman) |
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.
Typo: "Requesty model listing" seems incorrect. Possibly it should be "Request model listing"?
| - Fix: Requesty model listing (#7377 by @dtrugman, PR by @dtrugman) | |
| - Fix: Request model listing (#7377 by @dtrugman, PR by @dtrugman) |
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.
Thank you for your contribution! I've reviewed the changeset and have some suggestions for improvement that could be helpful for future changesets.
| @@ -0,0 +1,21 @@ | |||
| --- | |||
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.
Is this filename intentional? The PR title mentions v3.26.0, but the changeset file is named v3.25.24.md. Since this is a minor version bump, it would typically result in v3.26.0.
Could we consider renaming this to match the intended version?
| "roo-cline": minor | ||
| --- | ||
|
|
||
| - Sonic -> Grok Code Fast |
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.
Could we add more context here? It's unclear what type of change this represents - is this a rename, a migration, or something else?
| - Fix: Handle Mistral thinking content as reasoning chunks (#6842 by @Biotrioo, PR by @app/roomote) | ||
| - Fix: Make auto approve toggle trigger stay (#3909 by @kyle-apex, PR by @elianiva) | ||
| - Fix: Preserve user input when selecting follow-up choices (#7316 by @teihome, PR by @daniel-lxs) | ||
| - Fix: Resolve newTaskRequireTodos setting not working correctly (thanks @hannesrudolph!) |
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.
I notice there's inconsistent attribution formatting throughout the changeset:
Would it be better to standardize on one format for consistency? Also, several entries (lines 9, 12-19) are missing PR references which makes it harder to trace back to the actual code changes.
Important
Add changeset for v3.26.0 with bug fixes, new features, command handling improvements, and a dependency update.
newTaskRequireTodossetting functionality./initslash command.todosparameter in issue-fixer mode.rooignorechecks.drizzle-kitto v0.31.4.This description was created by
for beed9dc. You can customize this summary. It will automatically update as commits are pushed.