Skip to content

Conversation

@adegeo
Copy link
Contributor

@adegeo adegeo commented Dec 13, 2025

@adegeo adegeo requested a review from mikadumont December 13, 2025 00:54
@adegeo adegeo requested a review from a team as a code owner December 13, 2025 00:54
Copilot AI review requested due to automatic review settings December 13, 2025 00:54
@dotnetrepoman dotnetrepoman bot added this to the December 2025 milestone Dec 13, 2025
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 the GitHub Copilot app modernization documentation to reflect a new three-stage workflow (assessment, planning, and execution) for upgrading .NET applications. The changes provide clearer guidance on how developers interact with the modernization agent at each stage, including examples of the Markdown files generated during the process.

Key changes:

  • Restructured the upgrade workflow from a single-phase approach to a three-stage process with distinct assessment, planning, and execution phases
  • Updated examples to show the new Markdown documentation format generated at each stage (assessment.md, plan.md, tasks.md)
  • Revised custom upgrade instructions guidance to emphasize integration during the assessment stage

Reviewed changes

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

File Description
how-to-upgrade-with-github-copilot.md Rewrote to document the three-stage workflow with new section structure, updated examples, and clarified the role of each stage
how-to-custom-upgrade-instructions.md Updated to align custom upgrade instructions with the new three-stage workflow, emphasizing assessment-stage integration
visualstudio-copilot-upgrade5.png New or updated screenshot supporting the revised workflow documentation
visualstudio-copilot-upgrade6.png New or updated screenshot supporting the revised workflow documentation

Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

This LGTM, once you examine the Copilot comments. Some look reasonable.

I had one nit and one content question. Then, :shipit:

@adegeo adegeo merged commit a9b3553 into main Dec 15, 2025
11 checks passed
@adegeo adegeo deleted the adegeo/ghcp branch December 15, 2025 21:08
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.

2 participants