.NET 10 Migration - #36
Merged
Merged
Conversation
Co-authored-by: jamesmontemagno <1676321+jamesmontemagno@users.noreply.github.com>
Co-authored-by: jamesmontemagno <1676321+jamesmontemagno@users.noreply.github.com>
Co-authored-by: jamesmontemagno <1676321+jamesmontemagno@users.noreply.github.com>
Co-authored-by: jamesmontemagno <1676321+jamesmontemagno@users.noreply.github.com>
Co-authored-by: jamesmontemagno <1676321+jamesmontemagno@users.noreply.github.com>
Update repository to .NET 10
Member
Author
|
@copilot let's make the build validation to be smart to only build when something in the folder changes |
Contributor
|
@jamesmontemagno I've opened a new pull request, #37, to work on those changes. Once the pull request is ready, I'll request review from you. |
Co-authored-by: jamesmontemagno <1676321+jamesmontemagno@users.noreply.github.com>
Co-authored-by: jamesmontemagno <1676321+jamesmontemagno@users.noreply.github.com>
Add conditional execution to build validation workflow
|
dotnet10 |
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Validation skips AI/ML changes, while the devcontainer migration includes an invalid image tag and incompatible runtime setup.
Get a fresh assessment by requesting another Copilot review.
Review tier: Balanced
Findings: 4
Open findings (5)
What changed in this PR
Migrates the repository’s .NET samples and development environments to .NET 10 while adding automated build validation.
Changes:
- Retargets console, web, Blazor, IoT, AI/ML, MAUI, and hybrid samples to .NET 10.
- Updates Blazor render-mode APIs and fixes a Web API creation URI.
- Adds path-filtered build validation and updates devcontainer images.
| File | Description |
|---|---|
Visual Studio/.../RazorPagesCupcakes.csproj |
Targets .NET 10. |
Package Management with NuGet/.../AppLogger.csproj |
Targets .NET 10. |
Package Management with NuGet/.../Account.csproj |
Targets .NET 10. |
IoT/.../temperature.csproj |
Targets .NET 10. |
IoT/.../Adc.csproj |
Targets .NET 10. |
IoT/.../Sensor.csproj |
Targets .NET 10. |
Frontend Web with Blazor/.../BlazorMovieApp.csproj |
Targets .NET 10. |
Frontend Web with Blazor/.../BlazorApp.csproj |
Targets .NET 10. |
Frontend Web with Blazor/.../BlazorApp.Client.csproj |
Targets .NET 10. |
Frontend Web with Blazor/.../pages-routing-layouts/...csproj |
Targets .NET 10. |
Frontend Web with Blazor/.../introducing-blazor-components/...csproj |
Targets .NET 10. |
Frontend Web with Blazor/.../handling-events/...csproj |
Targets .NET 10. |
Frontend Web with Blazor/.../data-binding/...csproj |
Targets .NET 10. |
Containers/.../container-webapp.csproj |
Targets .NET 10. |
Blazor Hybrid/.../HybridTodoApp.csproj |
Updates MAUI targets to .NET 10. |
Blazor Hybrid/.../Website/Program.cs |
Uses current interactive-server APIs. |
Blazor Hybrid/.../Website.csproj |
Targets .NET 10. |
Blazor Hybrid/.../Counter.razor |
Uses the current render-mode directive. |
Blazor Hybrid/.../Website/Components/_Imports.razor |
Imports render-mode constants. |
Blazor Hybrid/.../Todo.razor |
Uses the current render-mode directive. |
Blazor Hybrid/.../Components.csproj |
Targets .NET 10. |
Blazor Hybrid/.../Components/_Imports.razor |
Imports render-mode constants. |
Backend Web with Web APIs/.../Program.cs |
Fixes the created-resource URI. |
Backend Web with Web APIs/.../MyNewApp.csproj |
Targets .NET 10. |
Artificial Intelligence and Machine Learning/.../MLModels.csproj |
Targets .NET 10. |
Artificial Intelligence and Machine Learning/.../GitHubIssuesWebApi.csproj |
Targets .NET 10. |
Artificial Intelligence and Machine Learning/.../chatgpt-plugin.csproj |
Targets .NET 10. |
Artificial Intelligence and Machine Learning/.../README.md |
Adds punctuation. |
.NET/.../MyPasswordGenerator.csproj |
Targets .NET 10. |
.NET/.../MyConsoleApp.csproj |
Targets .NET 10. |
.NET MAUI/.../MauiApp2.csproj |
Updates MAUI targets to .NET 10. |
.github/workflows/build-validation.yml |
Adds conditional sample builds. |
.devcontainer/base/devcontainer.json |
Updates the development image reference. |
.devcontainer/ai/Dockerfile |
Updates the AI environment to the .NET 10 SDK. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: jongalloway <68539+jongalloway@users.noreply.github.com>
jongalloway
approved these changes
Sep 11, 2026
Co-authored-by: jongalloway <68539+jongalloway@users.noreply.github.com>
Co-authored-by: jongalloway <68539+jongalloway@users.noreply.github.com>
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.