[CodeHealth] Fix clang-tidy warnings part 1#3493
Merged
marcalff merged 11 commits intoJun 24, 2025
Merged
Codecov / codecov/project
succeeded
Jun 24, 2025 in 1s
89.95% (+0.03%) compared to 1d41125
View this Pull Request on Codecov
89.95% (+0.03%) compared to 1d41125
Details
Codecov Report
Attention: Patch coverage is 75.00000% with 4 lines in your changes missing coverage. Please review.
Project coverage is 89.95%. Comparing base (
1d41125) to head (4480f10).
| Files with missing lines | Patch % | Lines |
|---|---|---|
| ...k/src/metrics/aggregation/lastvalue_aggregation.cc | 50.00% | 4 Missing |
Additional details and impacted files
@@ Coverage Diff @@
## main #3493 +/- ##
==========================================
+ Coverage 89.92% 89.95% +0.03%
==========================================
Files 219 219
Lines 7042 7039 -3
==========================================
- Hits 6332 6331 -1
+ Misses 710 708 -2 | Files with missing lines | Coverage Δ | |
|---|---|---|
| ...ry/sdk/metrics/aggregation/lastvalue_aggregation.h | 0.00% <ø> (ø) |
|
| ...elemetry/sdk/metrics/aggregation/sum_aggregation.h | 0.00% <ø> (ø) |
|
| sdk/src/logs/batch_log_record_processor.cc | 83.12% <100.00%> (ø) |
|
| sdk/src/logs/multi_log_record_processor.cc | 88.06% <100.00%> (+0.37%) |
⬆️ |
| sdk/src/logs/simple_log_record_processor.cc | 100.00% <100.00%> (ø) |
|
| sdk/src/metrics/aggregation/sum_aggregation.cc | 93.34% <ø> (+3.02%) |
⬆️ |
| sdk/src/metrics/state/metric_collector.cc | 93.23% <ø> (ø) |
|
| ...k/src/metrics/aggregation/lastvalue_aggregation.cc | 81.67% <50.00%> (-1.14%) |
⬇️ |
🚀 New features to boost your workflow:
- ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
Loading