Closed joegoldman2 closed 1 day ago
Attention: Patch coverage is 84.61538%
with 4 lines
in your changes missing coverage. Please review.
Project coverage is 77.93%. Comparing base (
71655ce
) to head (27d8ade
). Report is 346 commits behind head on main.
@ppittle, this PR is ready to be reviewed.
@joegoldman2 - thanks for your contribution!
@ppittle, @joegoldman2 - do you think that it is worth to document in changelog?
I would say no. I see the change as a implementation detail that should not change the behavior.
Changes
Prevent potential deadlocks by removing .Result calls in favour of
TaskFactory.StartNew(...)
.More context: https://github.com/open-telemetry/opentelemetry-dotnet-contrib/pull/1699#discussion_r1610995831
For significant contributions please make sure you have completed the following items:
CHANGELOG.md
updated for non-trivial changes