-
Notifications
You must be signed in to change notification settings - Fork 24
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
Update install-vscode.mdx #605
base: main
Are you sure you want to change the base?
Conversation
This update reflects the configuration parameters for the dev models according to the interface https://sourcegraph.com/github.com/sourcegraph/cody@main/-/blob/vscode/src/models/sync.ts?L98-105
@PriNova is attempting to deploy a commit to the Sourcegraph Team on Vercel. A member of the Team first needs to authorize it. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@aramaraju, would you like to take a quick review? :) |
@abeatrix can you bless this docs change? I'm not close to this code, unfortunately. |
Haven't worked closely with this part of the code, so I'm deferring to Beatrix here |
Needs a review for merging @MaedahBatool |
This PR can be closed since the variable option parameters of the llm-provider have been changed to map the providers endpoint API. Thank you |
This update reflects the configuration parameters for the dev models according to the interface https://sourcegraph.com/github.com/sourcegraph/cody@c6f6909f3e9683570a695ef6e096eccaa20f6aaf/-/blob/vscode/src/models/sync.ts?L102-110
If configured with
token
as input token, it will not work properly and set 7000 tokens as default.Pull Request approval
Although pull request approval is not enforced for this repository in order to reduce friction, merging without a review will generate a ticket for the docs team to review your changes. So if possible, have your pull request approved before merging.