Skip to content

Fix AppHost code example in EF Core migrations documentation - #126

Merged
David Pine (IEvangelist) merged 3 commits into
microsoft:mainfrom
lvb2104:fix/fix-efcore-apply-migration-docs
Dec 8, 2025
Merged

David Pine (IEvangelist) merged 3 commits into
microsoft:mainfrom
lvb2104:fix/fix-efcore-apply-migration-docs

Conversation

@ghost

Copy link
Copy Markdown

Fixed incorrect code example in the EF Core migrations documentation that was showing Worker service setup code instead of the proper AppHost distributed application configuration.

Changes Made

  • Corrected AppHost.cs code block: Replaced the incorrect Program.cs/Worker service content with the actual distributed application setup code showing how to properly wire up the migration service
  • Fixed code highlighting: Adjusted the line highlight range {6-21} → {7-9,13-14} to correctly highlight only the migration service configuration lines

- Corrected the AppHost.cs code block to show proper distributed application setup
- Fixed code highlighting to properly highlight the migration service configuration
Copilot AI review requested due to automatic review settings December 7, 2025 13:10

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

This PR corrects a critical documentation error in the EF Core migrations guide where the AppHost.cs code example was incorrectly showing Worker service setup code instead of distributed application configuration.

Key Changes:

  • Replaced incorrect AppHost.cs code example with proper distributed application setup showing SQL Server, migration service, and API project configuration
  • Updated syntax highlighting from {6-21} to {7-9,13-14} to focus on migration-specific configuration
  • Added syntax highlighting {6,9-10,12} to the Program.cs code block

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

…grations.mdx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…grations.mdx


docs: correct spacing in EF Core migration example

Co-authored-by: David Pine <david.pine@microsoft.com>
@ghost
ghost requested a review from David Pine (IEvangelist) December 7, 2025 17:26
@IEvangelist
David Pine (IEvangelist) merged commit b73ccaa into microsoft:main Dec 8, 2025
7 checks passed
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