Skip to content

Conversation

@lvb2104
Copy link
Contributor

@lvb2104 lvb2104 commented Dec 7, 2025

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
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 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


docs: correct spacing in EF Core migration example

Co-authored-by: David Pine <[email protected]>
@lvb2104 lvb2104 requested a review from IEvangelist December 7, 2025 17:26
@IEvangelist 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.

2 participants