Skip to content

feat: Logs are always enabled and SentryOptions.EnableLogs is ignored - #5504

Draft
jamescrosswell wants to merge 6 commits into
mainfrom
feat/remove-enable-logs-5479
Draft

feat: Logs are always enabled and SentryOptions.EnableLogs is ignored#5504
jamescrosswell wants to merge 6 commits into
mainfrom
feat/remove-enable-logs-5479

Revert "ref: Create a disabled structured logger for a disabled Hub"

47c133d
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 26, 2026 in 0s

74.78% (-0.09%) compared to 2be688c

View this Pull Request on Codecov

74.78% (-0.09%) compared to 2be688c

Details

Codecov Report

❌ Patch coverage is 88.88889% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 74.78%. Comparing base (2be688c) to head (47c133d).
⚠️ Report is 6 commits behind head on main.

Files with missing lines Patch % Lines
src/Sentry.Log4Net/SentryAppender.cs 50.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5504      +/-   ##
==========================================
- Coverage   74.87%   74.78%   -0.09%     
==========================================
  Files         513      513              
  Lines       18659    18749      +90     
  Branches     3636     3665      +29     
==========================================
+ Hits        13970    14022      +52     
- Misses       3819     3857      +38     
  Partials      870      870              

☔ 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.