Skip to content

docs: expand Azure Managed Identity BYOK guidance#1995

Open
scottaddie wants to merge 5 commits into
mainfrom
scottaddie-add-ami-doc-samples
Open

docs: expand Azure Managed Identity BYOK guidance#1995
scottaddie wants to merge 5 commits into
mainfrom
scottaddie-add-ami-doc-samples

Conversation

@scottaddie

@scottaddie scottaddie commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Summary

Directly addresses Patrick's comment: https://github.com/github/copilot-sdk-internal/issues/45#issuecomment-4971913761

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: c19bbd67-bd50-4f54-93e1-8c6961012916
@scottaddie scottaddie marked this pull request as ready for review July 15, 2026 19:56
@scottaddie scottaddie requested a review from a team as a code owner July 15, 2026 19:56
Copilot AI review requested due to automatic review settings July 15, 2026 19:56

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Expands Azure Managed Identity BYOK guidance across all SDK languages and documents on-demand bearer token providers.

Changes:

  • Adds static-token and callback examples for six SDKs.
  • Documents bearer token provider behavior in the BYOK reference.
Show a summary per file
File Description
docs/setup/azure-managed-identity.md Expands setup guidance and multi-language examples.
docs/auth/byok.md Adds bearer token provider documentation.

Review details

  • Files reviewed: 2/2 changed files
  • Comments generated: 20
  • Review effort level: Medium

Comment thread docs/setup/azure-managed-identity.md
Comment thread docs/setup/azure-managed-identity.md Outdated
Comment thread docs/setup/azure-managed-identity.md Outdated
Comment thread docs/setup/azure-managed-identity.md
Comment thread docs/auth/byok.md
Comment thread docs/setup/azure-managed-identity.md
Comment thread docs/setup/azure-managed-identity.md
Comment thread docs/setup/azure-managed-identity.md
Comment thread docs/setup/azure-managed-identity.md
Comment thread docs/setup/azure-managed-identity.md
Co-authored-by: scottaddie <10702007+scottaddie@users.noreply.github.com>
Comment thread .github/instructions/docs-style.instructions.md Outdated
Co-authored-by: scottaddie <10702007+scottaddie@users.noreply.github.com>
scottaddie and others added 2 commits July 15, 2026 15:31
@scottaddie scottaddie requested a review from sunbrye July 15, 2026 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants