Improve Azure OpenAI BYOK provider configuration - #45610
Conversation
Added new environment variables for model provider configuration, including bearer token, wire API, and Azure-specific settings.
Updated environment variable documentation for using custom models in GitHub Copilot CLI. Added Azure-specific variables and corrected formatting issues.
|
Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines. |
|
Hello, without any context I'm unable to process this PR for review—please feel free to comment here if this has been closed in error, and provide more context. Thank you |
|
Hey I have updated the above comment about the changes made in the doc and focusing on which issue. |
subatoi
left a comment
subatoi
left a comment
There was a problem hiding this comment.
Thank you, @Aditi16-ctrl—the issue in question hasn't been approved for a community contribution, but if/when it is, I'll try to include this contribution if possible
|
|
||
|
|
||
|
|
||
| The information contained in this communication is intended solely for the use of the individual or entity to whom it is addressed and others authorized to receive it. It may contain confidential or legally privileged information. If you are not the intended recipient you are hereby notified that any disclosure, copying, distribution or taking any action in reliance on the contents of this information is strictly prohibited and may be unlawful. If you have received this communication in error, please notify us immediately by responding to this email and then delete it from your system. The firm is neither liable for the proper and complete transmission of the information contained in this communication nor for any delay in its receipt. |
There was a problem hiding this comment.
I assume this passage has been added in error?
There was a problem hiding this comment.
Oh yes my bad, forgot to edit that out.
Just committed without the unnecessary note.
Thanks for the review, yes please try to include it in the contribution once approved.
Unnecessary note
2 participants
Why:
The BYOK documentation for GitHub Copilot CLI was missing several supported environment variables and contained an incorrect Azure provider base URL format. These updates improve the completeness and accuracy of the documentation and simplify the Azure configuration by removing the deployment-name dependency from the base URL.
Closes:
Closes #44864
What's being changed (if available, include any code snippets, screenshots, or gifs):
"https://YOUR-RESOURCE-NAME.openai.azure.com/openai/deployments/YOUR-DEPLOYMENT-NAME"
to:
"https://YOUR-RESOURCE-NAME.openai.azure.com"
Check off the following:
Additional context:
This PR directly addresses the documentation updates requested in #44864.
I made the requested changes to the affected BYOK documentation and raised this PR for review. If additional context is needed to process the PR, please feel free to comment here and I will provide it.