Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Update to .NET 9 SDK - #2003

#2003
Merged
martincostello merged 13 commits into
mainApp-vNext/Polly:mainfrom
dotnet-vnextApp-vNext/Polly:dotnet-vnextCopy head branch name to clipboard
Nov 12, 2024
Merged

Update to .NET 9 SDK#2003
martincostello merged 13 commits into
mainApp-vNext/Polly:mainfrom
dotnet-vnextApp-vNext/Polly:dotnet-vnextCopy head branch name to clipboard

Conversation

@martincostello

@martincostello martincostello commented Mar 1, 2024

Copy link
Copy Markdown
Member

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.
@codecov

codecov Bot commented Mar 1, 2024

Copy link
Copy Markdown

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.

Comment thread test/Polly.Core.Tests/Polly.Core.Tests.csproj Outdated
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
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.