Skip to content

Conversation

@vbreuss
Copy link
Member

@vbreuss vbreuss commented Jan 23, 2026

This PR removes .NET 6.0 from the test target frameworks to enable upgrading Microsoft.NET.Test.Sdk to version 18, which no longer supports .NET 6.0.

Key changes:

  • Removed net6.0 from the test target frameworks
  • Updated Microsoft.NET.Test.Sdk from version 17.14.0 to 18.0.1

@vbreuss vbreuss self-assigned this Jan 23, 2026
Copilot AI review requested due to automatic review settings January 23, 2026 15:28
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 PR removes .NET 6.0 from the test target frameworks to enable upgrading Microsoft.NET.Test.Sdk to version 18, which no longer supports .NET 6.0.

Changes:

  • Removed net6.0 from the test target frameworks
  • Updated Microsoft.NET.Test.Sdk from version 17.14.0 to 18.0.1

Reviewed changes

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

File Description
tests/Directory.Build.props Removed net6.0 from TargetFrameworks property
Directory.Packages.props Updated Microsoft.NET.Test.Sdk package version to 18.0.1

@vbreuss vbreuss enabled auto-merge (squash) January 23, 2026 15:29
@github-actions
Copy link

Test Results

    31 files  ±0      31 suites  ±0   18s ⏱️ -4s
 1 151 tests ±0   1 150 ✅ ±0    1 💤 ±0  0 ❌ ±0 
11 057 runs  ±0  10 218 ✅ ±0  839 💤 ±0  0 ❌ ±0 

Results for commit 5ca9f2d. ± Comparison against base commit d50f129.

This is necessary to update `Microsoft.NET.Test.Sdk` to v18
@vbreuss vbreuss force-pushed the topic/upgrade-net-test-sdk-v18 branch from 5ca9f2d to 5132e22 Compare January 23, 2026 15:34
@sonarqubecloud
Copy link

@vbreuss vbreuss merged commit 6d03e73 into main Jan 23, 2026
10 checks passed
@vbreuss vbreuss deleted the topic/upgrade-net-test-sdk-v18 branch January 23, 2026 15:39
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