Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to .NET 9 SDK #2003

Merged
merged 13 commits into from
Nov 12, 2024
Merged

Update to .NET 9 SDK #2003

merged 13 commits into from
Nov 12, 2024

Conversation

martincostello
Copy link
Member

@martincostello martincostello commented Mar 1, 2024

A long-lived PR for testing .NET 9 until the stable release in November 2024. net9.0 is explicitly not being added as a new TFM for the packages we ship to NuGet.org.

Changes include:

  • Update to the .NET 9 SDK.
  • Add usage of [DebuggerDisableUserUnhandledExceptions] to avoid newer versions of the Visual Studio debugger for breaking for exceptions on async code paths we are intentionally handling.
  • Target net8.0 and net9.0 in tests, samples and benchmarks.
  • Remove net6.0 target from tests, samples and benchmarks as it will both be out-of-support by November 2024.
  • Fix/suppress new IDE0022/S3236 warnings.
  • Only enable NuGet Audit for direct dependencies, not transient ones.

@martincostello martincostello added dependencies Pull requests that update a dependency file .NET Pull requests that update .NET code labels Mar 1, 2024
martincostello added a commit to martincostello/Polly that referenced this pull request Mar 1, 2024
- Fix CA1849 warning from updating snippets to .NET 8.
- Fix IDE0034 warning identified by the .NET 9 SDK in App-vNext#2003.
martincostello added a commit that referenced this pull request Mar 1, 2024
- Update various sample projects from `net7.0` to `net8.0`.
- Fix CA1849 warning from updating snippets to .NET 8.
- Fix IDE0034 warning identified by the .NET 9 SDK in #2003.
Copy link

codecov bot commented Mar 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.39%. Comparing base (14ac109) to head (c9c129e).
Report is 18 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2003      +/-   ##
==========================================
- Coverage   85.40%   85.39%   -0.01%     
==========================================
  Files         313      312       -1     
  Lines        7467     7464       -3     
  Branches     1125     1121       -4     
==========================================
- Hits         6377     6374       -3     
  Misses        745      745              
  Partials      345      345              
Flag Coverage Δ
linux 85.36% <100.00%> (-0.04%) ⬇️
macos 85.36% <100.00%> (-0.04%) ⬇️
windows 85.39% <100.00%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

martincostello added a commit to martincostello/Polly that referenced this pull request Apr 12, 2024
Fix IDE0057 warnings identified in App-vNext#2003.
martincostello added a commit that referenced this pull request Apr 12, 2024
Fix IDE0057 warnings identified in #2003.
@martincostello martincostello force-pushed the dotnet-vnext branch 2 times, most recently from bee5858 to eeac2c2 Compare May 15, 2024 11:22
@martincostello martincostello force-pushed the dotnet-vnext branch 2 times, most recently from f18d2e9 to ff96865 Compare May 29, 2024 10:42
@martincostello martincostello force-pushed the dotnet-vnext branch 2 times, most recently from 28ff68a to 9aa38cf Compare October 11, 2024 07:55
martincostello and others added 11 commits October 16, 2024 10:43
- Build with the .NET 9 (preview 6) SDK.
- Add tests TFM for `net9.0`.
- Remove tests for `net6.0` TFM.
- Run workflows on PRs to `dotnet-vnext` branch.
- Only audit direct NuGet dependencies.
- Fix IDE0022 warning.
- Temporarily disable some tests that are failing in GitHub Actions.
Disable TFM parallelism for `dotnet test` in the Polly.Core project on Windows to fix two failing tests.
See https://learn.microsoft.com/dotnet/core/whats-new/dotnet-9/sdk#run-tests-in-parallel.
Set the coverage threshold back to 100%.
Update .NET SDK to version 9.0.100-preview.7.24407.12.

---
updated-dependencies:
- dependency-name: Microsoft.NET.Sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: polly-updater-bot[bot] <138034000+polly-updater-bot[bot]@users.noreply.github.com>
Co-authored-by: polly-updater-bot[bot] <138034000+polly-updater-bot[bot]@users.noreply.github.com>
Update NuGet packages for .NET 9 preview 7.
Update Microsoft.Extensions.TimeProvider.Testing to .NET 9 preview 7.
Update .NET SDK to version 9.0.100-rc.1.24452.12.

---
updated-dependencies:
- dependency-name: Microsoft.NET.Sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: polly-updater-bot[bot] <138034000+polly-updater-bot[bot]@users.noreply.github.com>
Co-authored-by: polly-updater-bot[bot] <138034000+polly-updater-bot[bot]@users.noreply.github.com>
Update NuGet packages for RC1.
Update the TimeProvider testing package to preview 8.
Update .NET SDK to version 9.0.100-rc.2.24474.11.

---
updated-dependencies:
- dependency-name: Microsoft.NET.Sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: polly-updater-bot[bot] <138034000+polly-updater-bot[bot]@users.noreply.github.com>
Co-authored-by: polly-updater-bot[bot] <138034000+polly-updater-bot[bot]@users.noreply.github.com>
Bump NuGet packages for .NET 9 RC 2.
- Add usage of `[DebuggerDisableUserUnhandledExceptions]` to avoid newer versions of the Visual Studio debugger for breaking for exceptions on async code paths we are intentionally handling.
- Add copy of `[DebuggerDisableUserUnhandledExceptions]` for use in downlevel versions of .NET that do not contain the attribute.
@martincostello martincostello added this to the v8.5.0 milestone Oct 16, 2024
- Update to the stable version of .NET 9.
- Bump version to 8.5.0.
@martincostello martincostello marked this pull request as ready for review November 12, 2024 20:05
@martincostello martincostello merged commit 520ee38 into main Nov 12, 2024
20 checks passed
@martincostello martincostello deleted the dotnet-vnext branch November 12, 2024 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file .NET Pull requests that update .NET code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant