-
Notifications
You must be signed in to change notification settings - Fork 6.1k
Ignite update for .NET containerization. #49645
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Ignite update for .NET containerization. #49645
Conversation
There was a problem hiding this 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 pull request adds two screenshot images and updates the containerization documentation to improve the user workflow and provide additional guidance.
Key Changes
- Adds two PNG screenshot images to support the documentation (containerize-assessment-report.png and containerization-prompt.png)
- Enhances the containerization workflow with clearer prerequisites steps
- Improves wording for better clarity and consistency
- Adds an "Additional resources" section with a link to Containerization Assist
Reviewed Changes
Copilot reviewed 1 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| containerize-assessment-report.png | New screenshot showing the containerization task in the assessment report |
| containerization-prompt.png | New screenshot demonstrating how to start containerization using a prompt |
| containerization.md | Updates documentation with workflow improvements, wording changes, and additional resources |
| *Inspect my project and help me plan how to containerize my application using the #appmod-get-containerization-plan tool. | ||
| Execute the plan. | ||
| The end goal is to have Dockerfiles that are able to be built.* |
Copilot
AI
Nov 6, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nitpick] The multi-line prompt should be formatted as a single paragraph or properly indented as a code block. According to the coding guidelines, italics are used for single-line prompts. For multi-line prompts, consider using a code block format instead of italics.
| *Inspect my project and help me plan how to containerize my application using the #appmod-get-containerization-plan tool. | |
| Execute the plan. | |
| The end goal is to have Dockerfiles that are able to be built.* | |
| ``` | |
| Inspect my project and help me plan how to containerize my application using the #appmod-get-containerization-plan tool. | |
| Execute the plan. | |
| The end goal is to have Dockerfiles that are able to be built. | |
| ``` |
|
|
||
| 1. Fix the cloud readiness issues before running the **Docker Containerization** task. | ||
|
|
||
| 1. Make sure you enable **appModernizationDeploy** in the GitHub Copilot toolbox. |
Copilot
AI
Nov 6, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to the Markdown guidelines, avoid using weak phrases like 'make sure.' Instead, use imperative mood. Change to 'Enable appModernizationDeploy in the GitHub Copilot toolbox.'
Update doc for appmod containerization .NET
Internal previews