Update samples to use .net 8 (At least) - #574
Merged
Gordon Lam (yeelam-gordon) merged 8 commits intoNov 18, 2025
Merged
Conversation
Copilot started reviewing on behalf of
Gordon Lam (yeelam-gordon)
November 13, 2025 10:03
View session
Gordon Lam (yeelam-gordon)
requested a review
from Manodasan Wignarajah (manodasanW)
November 13, 2025 10:06
Contributor
There was a problem hiding this comment.
Pull Request Overview
This pull request modernizes the Windows App SDK sample projects by updating them from .NET 6 (which has reached end-of-life) to .NET 8, and standardizes runtime identifiers across all projects. The changes ensure compatibility with the latest .NET features and align with current naming conventions.
Key changes:
- Updated all C# project files from
net6.0-windows10.0.*tonet8.0-windows10.0.*target frameworks - Standardized RuntimeIdentifiers from
win10-x86/x64/arm64towin-x86/x64/arm64format - Enhanced the build script with conditional NuGet restore logic and parallel build support
Reviewed Changes
Copilot reviewed 44 out of 44 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| build.ps1 | Added conditional NuGet restore for packages.config-based projects, enabled parallel builds with /restore flag, and refactored elapsed time formatting |
| Samples/nuget.config | Added commented-out nuget.org source for local development reference |
| Multiple .csproj files (31 files) | Updated TargetFramework from net6.0 to net8.0 and standardized RuntimeIdentifiers from win10-* to win-* format |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Manodasan Wignarajah (manodasanW)
approved these changes
Nov 13, 2025
Gordon Lam (yeelam-gordon)
merged commit Nov 18, 2025
2cbefb5
into
release/experimental
4 checks passed
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.
Description
Given .net 6 already EOL, at least we should update all project to .net 8.
This pull request updates multiple C# sample projects to modernize their .NET framework versions and standardize runtime identifiers. The changes improve compatibility with newer .NET features and align runtime naming conventions across all samples.
.NET Framework and Runtime Modernization:
TargetFrameworkfor all sample.csprojfiles fromnet6.0-windows10.0.19041.0tonet8.0-windows10.0.19041.0, ensuring support for the latest .NET features and APIs. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17]RuntimeIdentifiersby replacingwin10-*identifiers withwin-*(e.g.,win10-x64→win-x64) across all projects for consistency and future-proofing. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17]These updates help keep the sample projects current and compatible with the latest development tools.
Target Release
Please specify which release this PR should align with 1.8
Checklist
Note that /azp run currently isn't working for this repo.