-
Notifications
You must be signed in to change notification settings - Fork 826
[WIP] Support .NET 10 #6307
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
base: main
Are you sure you want to change the base?
[WIP] Support .NET 10 #6307
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #6307 +/- ##
==========================================
- Coverage 86.76% 86.74% -0.02%
==========================================
Files 258 258
Lines 11879 11879
==========================================
- Hits 10307 10305 -2
- Misses 1572 1574 +2
Flags with carried forward coverage won't be shown. Click here to find out more. |
I don't know why the w3c-trace-context integration tests are failing. |
Caused by this breaking change - just need to work out it seems to be applying to .NET 8, 9 & 10 in the W3C tests and why manually setting it back isn't working. |
Fixed the .NET 10 tests - the .NET 8 and 9 tests are failing because the .NET 8 and 9 runtimes aren't present in the docker image, so it rolls forward to .NET 10, then the breaking change applies (but can't be turned off because the new API is missing), so the tests fail. |
a1fed39
to
097d765
Compare
This wasn't correct - the actual reason was that all TFMs were referencing the Might need tweaking depending on the outcome of #6327. |
Add support for building and testing with .NET 10, initially using .NET 10 preview 4.
Ensure the .NET 8 and 9 runtimes are present.
Use the .NET 10 SDK to build.
Remove obsolete attribute from compose file.
React to breaking change in .NET 10.
Remove leftover debug code.
Move the workaround to a static constructor so it runs early enough to affect the tests.
Update to preview 5 of .NET 10.
All TFMs are referencing the .NET 10 package, so the workaround needs applying to all of them.
f3d1c7b
to
d7b4e05
Compare
Improve developer experience when troubleshooting failing tests (like I was in open-telemetry#6307) by logging both stderr and stdout and ignoring warnings in the upstream Python tests.
Improve developer experience when troubleshooting failing tests (like I was in open-telemetry#6307) by logging both stderr and stdout and ignoring warnings in the upstream Python tests.
Improve developer experience when troubleshooting failing tests (like I was in open-telemetry#6307) by logging both stderr and stdout and ignoring warnings in the upstream Python tests.
Improve developer experience when troubleshooting failing tests (like I was in open-telemetry#6307) by logging both stderr and stdout and ignoring warnings in the upstream Python tests.
Changes
Add support for building and testing with .NET 10.
Initially using .NET 10 preview 4, will update as previews progress.
Merge requirement checklist
CHANGELOG.md
files updated for non-trivial changes