Skip to content

fix(logging): warn when ClearProviders() removes the logger providers UseSentry added - #5789

Merged
jamescrosswell merged 4 commits into
version7from
fix/warn-when-logger-providers-removed
Oct 27, 2026
Merged

jamescrosswell merged 4 commits into
version7from
fix/warn-when-logger-providers-removed

fix(logging): don't assume a Logging property in the warning

245cf66
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Oct 27, 2026 in 1s

76.21% (+0.03%) compared to 2194c92

View this Pull Request on Codecov

76.21% (+0.03%) compared to 2194c92

Details

Codecov Report

❌ Patch coverage is 93.33333% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 76.21%. Comparing base (2194c92) to head (245cf66).
⚠️ Report is 3 commits behind head on version7.

Files with missing lines Patch % Lines
...c/Sentry.Extensions.Logging/HostLoggerProviders.cs 90.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           version7    #5789      +/-   ##
============================================
+ Coverage     76.17%   76.21%   +0.03%     
============================================
  Files           526      528       +2     
  Lines         19265    19318      +53     
  Branches       3729     3737       +8     
============================================
+ Hits          14676    14724      +48     
- Misses         3734     3735       +1     
- Partials        855      859       +4     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.