Skip to content

Update tests and generator to .NET 10 - #562

Merged
jmaeagle99 merged 5 commits into
temporalio:mainfrom
jmaeagle99:net10
Nov 19, 2025
Merged

jmaeagle99 merged 5 commits into
temporalio:mainfrom
jmaeagle99:net10

Conversation

@jmaeagle99

@jmaeagle99 jmaeagle99 commented Nov 18, 2025

Copy link
Copy Markdown
Contributor

What was changed

  • Updated test projects and generator project to .NET 10.
  • Fixed code analysis violations and suppressed those that should not be fixed within workflows.
  • Adjusted standard library call tests due to changes in .NET 8+ runtime.

Why?

Lowest supported .NET version is 8 and both 8 and 9 will be out of support in less than a year at this point. Update to latest LTS (.NET 10) which will be supported for ~3 years.

Checklist

  1. Closes [Feature Request] Consider updating tests to use .NET 10 #522

  2. How was this tested: Local builds and test runs with only .NET 10 SDK installed. Built and ran all projects found to test the SDK tests, smoke test, benchmark, and docfx generation. Checked test counts before and after changes to check that tests were not dropped.

  3. Any docs updates needed? None required. Can consider if the Task.WhenAny -> Workflow.WhenAnyAsync recommendation should be adjusted (as well as the SDK implementation of Workflow.WhenAnyAsync) for .NET 8+.

@jmaeagle99
jmaeagle99 marked this pull request as ready for review November 18, 2025 20:01
@jmaeagle99
jmaeagle99 requested a review from a team as a code owner November 18, 2025 20:01

@cretz cretz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great, nothing blocking. But if possible, would like to see that "Build Package" workflow run inside this PR w/ a quick temporary update to confirm it works w/ the new containers/changes.

Comment thread .github/workflows/package.yml
Comment thread tests/Temporalio.Tests/KitchenSinkWorkflow.cs
@jmaeagle99
jmaeagle99 requested a review from cretz November 19, 2025 02:57

@cretz cretz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great, merge at will

Comment thread .github/workflows/package.yml
Comment thread tests/Temporalio.Tests/KitchenSinkWorkflow.cs
@jmaeagle99
jmaeagle99 merged commit d4a43f1 into temporalio:main Nov 19, 2025
10 checks passed
@jmaeagle99
jmaeagle99 deleted the net10 branch November 19, 2025 16:07
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.

[Feature Request] Consider updating tests to use .NET 10

2 participants