Skip to content

Conversation

@yeelam-gordon
Copy link
Contributor

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:

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.

Copy link

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 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.* to net8.0-windows10.0.* target frameworks
  • Standardized RuntimeIdentifiers from win10-x86/x64/arm64 to win-x86/x64/arm64 format
  • 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.

@yeelam-gordon yeelam-gordon merged commit 2cbefb5 into release/experimental Nov 18, 2025
4 checks passed
@yeelam-gordon yeelam-gordon deleted the user/yeelam/net628 branch November 18, 2025 03:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants