Skip to content

[CODE HEALTH] Fix clang-tidy warnings, part 2#3872

Merged
ThomsonTan merged 16 commits intoopen-telemetry:mainfrom
marcalff:tidy_special_functions
Mar 4, 2026
Merged

[CODE HEALTH] Fix clang-tidy warnings, part 2#3872
ThomsonTan merged 16 commits intoopen-telemetry:mainfrom
marcalff:tidy_special_functions

Conversation

@marcalff
Copy link
Copy Markdown
Member

@marcalff marcalff commented Feb 19, 2026

Fixes # (issue)

Changes

Please provide a brief description of the changes here.

  • Fix clang-tidy cppcoreguidelines-special-member-functions warnings in sdk and ext.

For significant contributions please make sure you have completed the following items:

  • CHANGELOG.md updated for non-trivial changes
  • Unit tests have been added
  • Changes in public API reviewed

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.07%. Comparing base (da8598c) to head (6ea74b6).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3872      +/-   ##
==========================================
+ Coverage   90.05%   90.07%   +0.03%     
==========================================
  Files         226      226              
  Lines        7200     7217      +17     
==========================================
+ Hits         6483     6500      +17     
  Misses        717      717              
Files with missing lines Coverage Δ
...clude/opentelemetry/sdk/common/atomic_unique_ptr.h 100.00% <ø> (ø)
...y/sdk/instrumentationscope/instrumentation_scope.h 100.00% <100.00%> (ø)
...pentelemetry/sdk/logs/batch_log_record_processor.h 100.00% <ø> (ø)
sdk/include/opentelemetry/sdk/logs/processor.h 100.00% <100.00%> (ø)
...pentelemetry/sdk/metrics/aggregation/aggregation.h 100.00% <100.00%> (ø)
...metry/sdk/metrics/aggregation/aggregation_config.h 89.48% <ø> (ø)
...e/opentelemetry/sdk/metrics/data/circular_buffer.h 100.00% <100.00%> (ø)
...nclude/opentelemetry/sdk/metrics/data/point_data.h 96.30% <100.00%> (+0.15%) ⬆️
...opentelemetry/sdk/metrics/export/metric_producer.h 100.00% <100.00%> (ø)
.../include/opentelemetry/sdk/metrics/metric_reader.h 50.00% <ø> (ø)
... and 20 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@marcalff marcalff marked this pull request as ready for review February 21, 2026 20:51
@marcalff marcalff requested a review from a team as a code owner February 21, 2026 20:51
@marcalff marcalff marked this pull request as draft February 22, 2026 14:11
@marcalff marcalff marked this pull request as ready for review February 27, 2026 23:39
@marcalff marcalff added the pr:please-review This PR is ready for review label Feb 28, 2026
Comment thread ext/include/opentelemetry/ext/http/server/http_server.h Outdated
@ThomsonTan ThomsonTan merged commit 9fcfe9d into open-telemetry:main Mar 4, 2026
68 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:please-review This PR is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants