Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Oct 3, 2025

This PR updates the project dependencies to use the latest stable versions of .NET and C# while maintaining compatibility and ensuring no tests break.

Changes

Updated three key configuration files to modernize the .NET stack:

  • global.json: SDK version updated from 8.0.09.0.0
  • Directory.Build.props: LangVersion updated from 12.013.0 (C# 13)
  • Common.props: TargetFrameworks updated from net8.0;net7.0;net6.0net9.0;net8.0

Updated documentation to reflect the new versions:

  • README.md: Updated requirements section from ".NET 8.0, 7.0, and 6.0" to ".NET 9.0 and 8.0"
  • Errata.md: Updated title from "Essential C# 12.0" to "Essential C# 13.0"
  • .github/actions/setup-dotnet/action.yml: Updated dotnet-version list from 6.0.x, 7.0.x, 8.0.x to 8.0.x, 9.0.x

Rationale

  • Removes EOL framework warnings: .NET 6.0 and 7.0 are out of support and were generating NETSDK1138 warnings
  • Enables C# 13.0 features: Projects can now use the latest C# language features
  • Reduces multi-targeting overhead: Simplified from 3 to 2 target frameworks while maintaining modern platform support
  • Maintains compatibility: Kept .NET 8.0 as a secondary target for broader compatibility

Validation

Build: Solution builds successfully with 0 errors
Tests: All tests pass across both target frameworks (net9.0 and net8.0)
Warnings: No new warnings introduced; only expected warnings remain (intentional #warning directives and SYSLIB warnings for code examples)
SDK: Verified using .NET 9.0.305 SDK

All existing tests continue to pass, confirming no breaking changes were introduced.

Original prompt

Update the .net dependencies as much as we can without

  1. Breaking out tests
  2. Causing any build warnings or errors

You can update TFM's and LangVersion (and global.json) - but only to C# 13.0 and it's compatible .net version (I think .net 9.0?)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title Update the .net dependencies as much as we can without 1. Breaking out tests 2. Causing any build warnings or errors You can update TFM's and LangVersion (and global.json) - but only to C# 13.0 and it's compatible .net version (I think .net 9.0?) Update .NET dependencies to C# 13.0 and .NET 9.0 Oct 3, 2025
@BenjaminMichaelis BenjaminMichaelis marked this pull request as ready for review October 3, 2025 10:53
Copilot AI review requested due to automatic review settings October 3, 2025 10:53
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

Updates .NET dependencies to use the latest stable versions (C# 13.0 and .NET 9.0) while removing end-of-life framework targets to eliminate warnings and modernize the codebase.

  • Updated SDK version from 8.0.0 to 9.0.0
  • Upgraded C# language version from 12.0 to 13.0
  • Simplified target frameworks from net8.0;net7.0;net6.0 to net9.0;net8.0

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
global.json Updates .NET SDK version to 9.0.0 for latest tooling
Directory.Build.props Enables C# 13.0 language features across all projects
Common.props Removes EOL frameworks and targets .NET 9.0 and 8.0

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@BenjaminMichaelis
Copy link
Member

@copilot did we update any relevant documentation and similar?

Copy link
Contributor Author

Copilot AI commented Oct 3, 2025

@copilot did we update any relevant documentation and similar?

Yes, I've updated the documentation in commit f5f79e2:

  • README.md: Updated requirements section to reflect .NET 9.0 and 8.0
  • Errata.md: Updated title from "Essential C# 12.0" to "Essential C# 13.0"
  • .github/actions/setup-dotnet/action.yml: Updated dotnet-version list from 6.0.x/7.0.x/8.0.x to 8.0.x/9.0.x

@BenjaminMichaelis BenjaminMichaelis merged commit 2826269 into v13.0 Oct 4, 2025
6 checks passed
@BenjaminMichaelis BenjaminMichaelis deleted the copilot/fix-6fc91057-f32e-4652-8ffc-62c6e8f96e52 branch October 4, 2025 04:27
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.

2 participants