-
Couldn't load subscription status.
- Fork 2.4k
chore: add changeset for v3.28.2 #7979
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
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 adding the changeset for v3.28.2! The release notes are comprehensive and well-documented. I've left a few minor suggestions for consistency that you might want to consider.
|  | ||
|
|
||
| - Improve auto-approve UI with smaller and more subtle design (thanks @brunobergher!) | ||
| - Fix: Message queue re-queue loop in Task.ask() causing performance issues (#7861 by @hannesrudolph, PR by @daniel-lxs) |
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 noticed the attribution format varies throughout the changeset. Some entries use "thanks @username!" while others use "by @username, PR by @username". Would it be helpful to standardize this for consistency?
For example, you could use:
| - Fix: Restrict @-mention parsing to line-start or whitespace boundaries to prevent false triggers (#7875 by @hannesrudolph, PR by @app/roomote) | ||
| - Fix: Make nested git repository warning persistent with path info for better visibility (#7884 by @hannesrudolph, PR by @app/roomote) | ||
| - Fix: Include API key in Ollama /api/tags requests for authenticated instances (#7902 by @ItsOnlyBinary, PR by @app/roomote) | ||
| - Fix: Preserve original first message context during conversation condensing (thanks @daniel-lxs!) |
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.
For consistency with the other fixes that reference issue numbers, do you happen to know if there's an associated issue number for this fix? It would be helpful to include it if available.
| "roo-cline": patch | ||
| --- | ||
|
|
||
|  |
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 alt text mentions "Auto-approve improvements" but the changeset includes many other changes. Would a more generic alt text like "3.28.2 Release" or "3.28.2 Release Notes" better represent all the changes in this version?
Important
The v3.28.2 changeset includes UI improvements, bug fixes, and new features, enhancing performance and security.
Task.ask()causing performance issues.@-mentionparsing to line-start or whitespace to prevent false triggers./api/tagsrequests for authenticated instances..changeset/v3.28.2.mdto document these changes.This description was created by
for 02201d5. You can customize this summary. It will automatically update as commits are pushed.