-
Notifications
You must be signed in to change notification settings - Fork 2.6k
More changes #7232
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
More changes #7232
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| --- | ||
| "roo-cline": patch | ||
| --- | ||
|
|
||
| - Fix: Add missing zaiApiKey and doubaoApiKey to SECRET_STATE_KEYS (#7082 by @app/roomote) | ||
|
Contributor
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. I notice this changeset only documents the changes but doesn't include the actual implementation. Were the code changes for adding |
||
| - Feat: Add new models and update configurations for vscode-lm (thanks @NaccOll!) | ||
|
Contributor
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. Could we be more specific about which models were added and what configurations were updated? This level of detail would be helpful for debugging or if we need to rollback specific changes in the future. |
||
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 attribution pattern intentional? The fix references issue #7082 which was created by @jayxuz, but credits @app/roomote. Would it be clearer to separate issue reporter from PR author attribution, something like: "(#7082 reported by @jayxuz, fixed by @app/roomote)"?