Skip to content

Conversation

@jidddddd
Copy link
Contributor

@jidddddd jidddddd commented Nov 6, 2025

Update doc for appmod containerization .NET


Internal previews

📄 File 🔗 Preview link
docs/azure/migration/appmod/containerization.md docs/azure/migration/appmod/containerization

Copilot AI review requested due to automatic review settings November 6, 2025 07:12
@jidddddd jidddddd requested review from a team and alexwolfmsft as code owners November 6, 2025 07:12
@dotnetrepoman dotnetrepoman bot added this to the November 2025 milestone Nov 6, 2025
@dotnet-policy-service dotnet-policy-service bot added dotnet-azure/svc community-contribution Indicates PR is created by someone from the .NET community. labels Nov 6, 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 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

Comment on lines +49 to +51
*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.*
Copy link

Copilot AI Nov 6, 2025

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.

Suggested change
*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.
```

Copilot uses AI. Check for mistakes.

1. Fix the cloud readiness issues before running the **Docker Containerization** task.

1. Make sure you enable **appModernizationDeploy** in the GitHub Copilot toolbox.
Copy link

Copilot AI Nov 6, 2025

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

Copilot generated this review using guidance from repository custom instructions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution Indicates PR is created by someone from the .NET community. dotnet-azure/svc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant