Skip to content

Conversation

@KaptenJon
Copy link

@KaptenJon KaptenJon commented Dec 11, 2025

I added .net 10, upgraded nugets, removed deprecated aspnetcore3.1 (supported partly by netstandard) and fixed warnings from stylecop.

Copilot AI review requested due to automatic review settings December 11, 2025 17:02
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 attempts to add .NET 10 as a target framework for the SoapCore library and its related projects. However, there is a critical issue: .NET 10 does not exist yet as of December 2025. The latest version of .NET is .NET 9, which was released in November 2024. Microsoft follows an annual release cadence, so .NET 10 would not be expected until November 2025 at the earliest.

Key Changes

  • Added net10.0 as a target framework across multiple project files (SoapCore, tests, benchmarks)
  • Updated package references to use version 10.x packages that do not exist
  • Created new .NET 10 sample projects (Net10Server and Net10Client)
  • Updated Visual Studio solution file to reference non-existent Visual Studio 18

Reviewed changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
src/SoapCore/SoapCore.csproj Adds net10.0 target framework and references non-existent v10 packages
src/SoapCore.Tests/SoapCore.Tests.csproj Adds net10.0 target with non-existent v10 test packages
src/SoapCore.Benchmark/SoapCore.Benchmark.csproj Adds net10.0 target for benchmarks with invalid package versions
samples/Sample.sln Updated to reference non-existent Visual Studio 18 and includes new .NET 10 projects
samples/README.md Documentation referencing non-existent .NET 10
samples/Net10Server/* New sample server project targeting non-existent net10.0 framework
samples/Net10Client/* New sample client project targeting non-existent net10.0 framework
samples/Net10Sample/README.md Documentation for non-existent .NET 10 sample
.gitignore Added lutconfig file to ignore list

Critical Issue: All references to .NET 10 (net10.0) and version 10.x packages should be changed to .NET 9 (net9.0) and appropriate version 9.x packages to target the actual latest .NET version.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@KaptenJon
Copy link
Author

You say that .net 10 does not exists. But it really does it was released november 2025.

Pull request overview

This PR attempts to add .NET 10 as a target framework for the SoapCore library and its related projects. However, there is a critical issue: .NET 10 does not exist yet as of December 2025. The latest version of .NET is .NET 9, which was released in November 2024. Microsoft follows an annual release cadence, so .NET 10 would not be expected until November 2025 at the earliest.

Key Changes

  • Added net10.0 as a target framework across multiple project files (SoapCore, tests, benchmarks)
  • Updated package references to use version 10.x packages that do not exist
  • Created new .NET 10 sample projects (Net10Server and Net10Client)
  • Updated Visual Studio solution file to reference non-existent Visual Studio 18

Reviewed changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 7 comments.

Show a summary per file
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

Copilot reviewed 41 out of 43 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

KaptenJon and others added 3 commits December 12, 2025 10:44
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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

Copilot reviewed 41 out of 43 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

Copilot reviewed 41 out of 43 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

Copilot reviewed 41 out of 43 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

1 participant