Skip to content

Move convert values to internal/shared/logutil

Codecov / codecov/project succeeded Oct 14, 2024 in 0s

67.0% (+0.5%) compared to c3c8538

View this Pull Request on Codecov

67.0% (+0.5%) compared to c3c8538

Details

Codecov Report

Attention: Patch coverage is 99.03537% with 3 lines in your changes missing coverage. Please review.

Project coverage is 67.0%. Comparing base (c3c8538) to head (74ac4bf).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
bridges/otellogr/internal/logutil/convert.go 99.0% 1 Missing ⚠️
bridges/otellogrus/internal/logutil/convert.go 99.0% 1 Missing ⚠️
bridges/otelzap/internal/logutil/convert.go 99.0% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main   #6237     +/-   ##
=======================================
+ Coverage   66.5%   67.0%   +0.5%     
=======================================
  Files        188     190      +2     
  Lines      12456   12576    +120     
=======================================
+ Hits        8284    8428    +144     
+ Misses      3879    3858     -21     
+ Partials     293     290      -3     
Files with missing lines Coverage Δ
bridges/otellogr/logsink.go 98.2% <100.0%> (ø)
bridges/otellogrus/hook.go 97.5% <100.0%> (-0.1%) ⬇️
bridges/otelzap/encoder.go 100.0% <100.0%> (+13.8%) ⬆️
bridges/otellogr/internal/logutil/convert.go 99.0% <99.0%> (ø)
bridges/otellogrus/internal/logutil/convert.go 99.0% <99.0%> (ø)
bridges/otelzap/internal/logutil/convert.go 99.0% <99.0%> (ø)

... and 1 file with indirect coverage changes