Skip to content

Clamp minsev Severity.Severity() conversion#7748

Merged
MrAlias merged 9 commits into
open-telemetry:mainfrom
MrAlias:clamp-sev
Aug 24, 2025
Merged

Clamp minsev Severity.Severity() conversion#7748
MrAlias merged 9 commits into
open-telemetry:mainfrom
MrAlias:clamp-sev

Conversation

@MrAlias
Copy link
Copy Markdown
Contributor

@MrAlias MrAlias commented Aug 14, 2025

Resolve #7662

@MrAlias MrAlias added this to the v1.38.0 milestone Aug 14, 2025
@MrAlias MrAlias marked this pull request as ready for review August 14, 2025 18:35
@MrAlias MrAlias requested a review from a team as a code owner August 14, 2025 18:35
@pellared
Copy link
Copy Markdown
Member

CC @sahidvelji

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.9%. Comparing base (0ce0b63) to head (503cb3f).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #7748   +/-   ##
=====================================
  Coverage   78.9%   78.9%           
=====================================
  Files        186     186           
  Lines      14819   14823    +4     
=====================================
+ Hits       11699   11703    +4     
  Misses      2780    2780           
  Partials     340     340           
Files with missing lines Coverage Δ
processors/minsev/severity.go 100.0% <100.0%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@MrAlias MrAlias merged commit 09874df into open-telemetry:main Aug 24, 2025
35 checks passed
@MrAlias MrAlias deleted the clamp-sev branch August 24, 2025 19:29
@MrAlias MrAlias mentioned this pull request Aug 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clamp "minsev".Severity.Severity() to Defined Range of log.Severity

4 participants