Skip to content

Update for .NET 11 - #428

Merged
vcsjones merged 3 commits into
mainfrom
net11-update
Nov 12, 2026
Merged

vcsjones merged 3 commits into
mainfrom
net11-update

Conversation

@vcsjones

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI balanced review requested due to automatic review settings November 12, 2026 16:38

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The build script still references removed net8.0 assets, causing signing and copying failures and leaving the new net10.0 and net11.0 assemblies unsigned.

1 open issue
What changed in this PR

Updates AzureSignTool for .NET 11, including framework targets, dependencies, and test infrastructure.

Changes:

  • Targets the tool at .NET 10 and 11, and the core library at .NET Standard 2.0 and .NET 10.
  • Migrates both test projects to .NET 11 and xUnit’s Microsoft.Testing.Platform integration.
  • Updates central package versions and the PR workflow’s SDK.
File Description
test/​AzureSignTool.Tests/​AzureSignTool.Tests.csproj Moves tool tests to .NET 11 and the new test runner.
test/​AzureSign.Core.Tests/​AzureSign.Core.Tests.csproj Moves core tests to .NET 11 and the new test runner.
src/​AzureSignTool/​AzureSignTool.csproj Updates tool targets and removes the cryptography compatibility dependency.
src/​AzureSign.Core/​AzureSign.Core.csproj Replaces the .NET 8 target with .NET 10 and updates the AOT condition.
global.json Selects Microsoft.Testing.Platform for test execution.
Directory.Packages.props Updates dependencies and replaces legacy test packages.
.github/​workflows/​pr.yml Selects the .NET 11 SDK for PR tests.

Review effort: Balanced

@vcsjones
vcsjones merged commit 491a2b6 into main Nov 12, 2026
5 of 6 checks passed
@vcsjones
vcsjones deleted the net11-update branch November 12, 2026 16:58
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