upgrade to .NET 10; upgrade packages#434
Open
xj-ms wants to merge 1 commit into
Open
Conversation
xj-ms
requested review from
helenkzhang,
jmaxson-ms,
malavshah11 and
trgibeau
as code owners
There was a problem hiding this comment.
Pull request overview
Updates the repository’s tool/test target frameworks and several dependency versions as part of moving to .NET 10, while keeping .NET 8 and existing .NET Standard / netcoreapp3.1 targets.
Changes:
- Switch multiple tools and test projects from
net8.0tonet10.0and addnet10.0to the Runtime.NetCoreApp multi-targeting list. - Upgrade several NuGet dependencies (e.g.,
Microsoft.Extensions.*,System.Text.Json,Newtonsoft.Json.Schema,System.Reflection.Emit,NuGet.Versioning). - Update Nerdbank.GitVersioning to a newer version via
Directory.Build.props.
Reviewed changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/Tools/PlugInConfigurationEditor/PlugInConfigurationEditor.csproj | Move tool to net10.0 and bump System.CommandLine beta version. |
| src/PluginsSystem/Tools/Microsoft.Performance.Toolkit.Plugins.Cli/Microsoft.Performance.Toolkit.Plugins.Cli.csproj | Upgrade CLI dependencies (notably Microsoft.Extensions.* and Newtonsoft packages). |
| src/PluginsSystem/Tools/Microsoft.Performance.Toolkit.Plugins.Cli.Tests/Microsoft.Performance.Toolkit.Plugins.Cli.Tests.csproj | Move CLI tests to net10.0. |
| src/PluginsSystem/Microsoft.Performance.Toolkit.Plugins.Runtime/Microsoft.Performance.Toolkit.Plugins.Runtime.csproj | Upgrade DistributedLock.FileSystem. |
| src/PluginsSystem/Microsoft.Performance.Toolkit.Plugins.Runtime.Tests/Microsoft.Performance.Toolkit.Plugins.Runtime.Tests.csproj | Move runtime tests to net10.0. |
| src/PluginsSystem/Microsoft.Performance.Toolkit.Plugins.Core.Tests/Microsoft.Performance.Toolkit.Plugins.Core.Tests.csproj | Move core tests to net10.0 (plus encoding/BOM change). |
| src/Microsoft.Performance.Toolkit.Engine/Microsoft.Performance.Toolkit.Engine.csproj | Upgrade System.Reflection.Emit. |
| src/Microsoft.Performance.Toolkit.Engine.Tests/Microsoft.Performance.Toolkit.Engine.Tests.csproj | Move engine tests to net10.0. |
| src/Microsoft.Performance.Toolkit.Engine.Tests.Driver/Microsoft.Performance.Toolkit.Engine.Tests.Driver.csproj | Move test driver app to net10.0. |
| src/Microsoft.Performance.SDK/Microsoft.Performance.SDK.csproj | Upgrade NuGet.Versioning (staying on a netstandard-compatible major) and System.Reflection.Emit. |
| src/Microsoft.Performance.SDK.Tests/Microsoft.Performance.SDK.Tests.csproj | Move SDK tests to net10.0. |
| src/Microsoft.Performance.SDK.Runtime.Tests/Microsoft.Performance.SDK.Runtime.Tests.csproj | Move runtime tests to net10.0. |
| src/Microsoft.Performance.SDK.Runtime.NetCoreApp/Microsoft.Performance.SDK.Runtime.NetCoreApp.csproj | Drop net6.0 and add net10.0 to multi-targeting; upgrade System.Reflection.MetadataLoadContext. |
| src/Microsoft.Performance.SDK.Runtime.NetCoreApp.Tests/Microsoft.Performance.SDK.Runtime.NetCoreApp.Tests.csproj | Move tests to net10.0, upgrade Roslyn package, and add CopyLocalLockFileAssemblies. |
| src/Directory.Build.targets | Central package update for System.Text.Json to 10.0.8. |
| src/Directory.Build.props | Upgrade Nerdbank.GitVersioning package version. |
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.
Just upgrading the framework to .NET 10.
Support for .NET 6 is dropped while .NET 8 support is kept.
No changes to .NET Standard and .NET Core 3.1 support
Certain Nuget packages have also been also upgraded to new versions