Migration - 2 of several #5748
Open
Migration - 2 of several #5748
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Deleted favicon.png and AspireApp.sln files as part of cleanup. - Removed write-your-first-test.md documentation and updated related links in toc.yml and hosting-startup-not-supported.md to point to the new Aspire testing overview. - Updated links in the what's new documentation for Aspire 9.1 to reflect the new testing documentation structure.
…pire diagnostics overview.
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR is part 2 of a migration effort focused on reorganizing documentation related to testing, integrations (including community toolkit), and diagnostics. The changes remove a substantial amount of Azure-related documentation and supporting files, along with authentication-related content and examples.
Key Changes
- Removal of Azure integration documentation and examples
- Deletion of authentication-related content (particularly Keycloak integration)
- Removal of multiple snippet projects and associated files
Reviewed Changes
Copilot reviewed 78 out of 776 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| docs/azure/*.md | Removed several Azure integration overview and configuration documents |
| docs/azure/snippets/* | Deleted complete Azure snippet projects including Bicep, ACR, and ACA examples |
| docs/authentication/keycloak-integration.md | Removed comprehensive Keycloak authentication integration documentation |
| docs/authentication/snippets/AspireApp/* | Deleted complete AspireApp authentication example project |
…grations - Deleted favicon.png and Dapr.sln files from the Dapr snippet. - Removed SQLite Entity Framework integration documentation. - Removed SQLite integration documentation. - Updated links in local-auth-disabled-for-azure-resources.md for Azure Web PubSub. - Corrected Azure Container Apps documentation link in aca-deployment-aspire-cli.md. - Updated Azure Key Vault and managed identity links in azure-security-best-practices.md. - Cleaned up caching integrations deployment documentation. - Revised SQL Server integration deployment documentation. - Updated test-deployed-app.md to fix link formatting. - Adjusted dev-containers.md to remove unnecessary Docker reference.
adegeo
approved these changes
Nov 18, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary: Migration
Internal previews
Toggle expand/collapse