-
Notifications
You must be signed in to change notification settings - Fork 803
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 currently supported Target Frameworks. #2331
Conversation
This drops support for end-of-life TFMs: anything before `net8.0` keeping `netstandard2.0` in any library that supported it before. Tests are run on net8 and net9.
This already happens in the root test Directory.Build.props file.
…t9 tests can't run in parallel. Also remove version from docker-compose, since it is obsolete.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do tests have target net9
and net8
, but libraries only have net8
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you @eerhardt !
test/HealthChecks.Elasticsearch.Tests/HealthChecks.Elasticsearch.Tests.csproj
Show resolved
Hide resolved
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #2331 +/- ##
==========================================
+ Coverage 64.25% 66.88% +2.62%
==========================================
Files 248 268 +20
Lines 8399 8730 +331
Branches 590 631 +41
==========================================
+ Hits 5397 5839 +442
+ Misses 2853 2723 -130
- Partials 149 168 +19
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
What this PR does / why we need it:
This drops support for end-of-life TFMs: anything before
net8.0
keepingnetstandard2.0
in any library that supported it before.Tests are run on net8 and net9.
Does this PR introduce a user-facing change?:
Yes, it drops support for end-of-life TFMs in preparation for releasing the next major version.
Please make sure you've completed the relevant tasks for this PR, out of the following list: