Skip to content

Update language agnostic includes to use cross repo includes#52301

Merged
gewarren merged 20 commits intomainfrom
include-updates
Mar 20, 2026
Merged

Update language agnostic includes to use cross repo includes#52301
gewarren merged 20 commits intomainfrom
include-updates

Conversation

@alexwolfmsft
Copy link
Contributor

@alexwolfmsft alexwolfmsft commented Mar 13, 2026

@alexwolfmsft alexwolfmsft marked this pull request as ready for review March 16, 2026 16:07
Copilot AI review requested due to automatic review settings March 16, 2026 16:07
@alexwolfmsft alexwolfmsft requested review from a team, gewarren and scottaddie as code owners March 16, 2026 16:07
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates Azure SDK authentication documentation to reference centralized cross-repo include files, aiming to standardize shared authentication content and reduce duplication.

Changes:

  • Swaps local include paths in Azure authentication articles to ~/azure-dev-docs-pr/articles/includes/authentication/....
  • Replaces several long, inline “sign-in” instruction blocks with centralized include files.
  • Updates one link in an AI quickstart include to a docset-root-relative /dotnet/... URL.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
docs/azure/sdk/authentication/user-assigned-managed-identity.md Switches managed identity concepts include to centralized cross-repo include.
docs/azure/sdk/authentication/system-assigned-managed-identity.md Switches managed identity concepts include to centralized cross-repo include.
docs/azure/sdk/authentication/on-premises-apps.md Switches app registration include to centralized cross-repo include.
docs/azure/sdk/authentication/local-development-service-principal.md Switches multiple setup includes (app registration, group, role assignment) to centralized cross-repo includes.
docs/azure/sdk/authentication/local-development-dev-accounts.md Switches group/role includes and replaces tool sign-in content with centralized includes.
docs/azure/sdk/authentication/local-development-broker.md Switches broker-related includes to centralized cross-repo includes.
docs/ai/quickstarts/includes/ai-templates-azure-openai.md Updates one internal link to a docset-root-relative /dotnet/... link.

You can also share your feedback on Copilot code review. Take the survey.

@alexwolfmsft alexwolfmsft requested a review from jsquire as a code owner March 16, 2026 19:05
alexwolfmsft and others added 2 commits March 16, 2026 17:14
Co-authored-by: Scott Addie <10702007+scottaddie@users.noreply.github.com>
Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
@gewarren
Copy link
Contributor

MSDocs Build Verifier error is a false positive: dotnet/docs-tools#641

@alexwolfmsft alexwolfmsft enabled auto-merge (squash) March 20, 2026 16:27
@alexwolfmsft alexwolfmsft disabled auto-merge March 20, 2026 16:27
@gewarren gewarren merged commit 863ac42 into main Mar 20, 2026
10 of 11 checks passed
@gewarren gewarren deleted the include-updates branch March 20, 2026 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants