-
Notifications
You must be signed in to change notification settings - Fork 1
Add CICD Pipeline + Rev to .NET 10 #3
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?
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 attempts to upgrade the codebase from .NET 9 to .NET 10 and adds a comprehensive CI/CD pipeline using GitHub Actions. However, the upgrade targets a version of .NET that does not exist as of January 2025.
Key changes:
- Upgrade all project target frameworks from net9.0 to net10.0
- Update SDK version to 10.0.100 across build configurations
- Remove redundant package references (System.Net.Http, System.Security.Cryptography.X509Certificates)
- Add new GitHub Actions workflow for build, test, and deployment to Azure Container Apps
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/Microsoft.TryDotNet/Microsoft.TryDotNet.csproj | Updates target framework to net10.0 and removes redundant package references |
| src/Microsoft.TryDotNet.WasmRunner/Microsoft.TryDotNet.WasmRunner.csproj | Updates target framework to net10.0 |
| src/Microsoft.TryDotNet.Tests/Microsoft.TryDotNet.Tests.csproj | Updates target framework to net10.0 and removes redundant package references |
| src/Microsoft.TryDotNet.SimulatorGenerator/Microsoft.TryDotNet.SimulatorGenerator.csproj | Updates target framework to net10.0 and removes redundant package reference |
| src/Microsoft.TryDotNet.IntegrationTests/Microsoft.TryDotNet.IntegrationTests.csproj | Updates target framework to net10.0 and removes redundant package references |
| global.json | Updates SDK and dotnet tool versions to 10.0.100 |
| eng/build.ps1 | Updates playwright path reference from net9.0 to net10.0 |
| azure-pipelines.yml | Updates SDK installation version to 10.0.100 |
| azure-pipelines-CI.yml | Updates SDK installation version to 10.0.100 |
| Dockerfile | Updates Docker base images from dotnet/sdk:9.0 to 10.0 |
| Directory.Packages.props | Updates ASP.NET Core and System.Drawing.Common package versions to 10.0.0 |
| .github/workflows/Build-Test-And-Deploy.yaml | Adds new GitHub Actions workflow for CI/CD with build, test, containerization, and Azure deployment |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Description
Added CICD pipeline to validate PRs and push container image to registry and deploy Container App.
Fixes #Issue_Number (if available)
Ensure that your pull request has followed all the steps below: