opentelemetry-cpp icon indicating copy to clipboard operation
opentelemetry-cpp copied to clipboard

[CI] upgrade to clang-tidy 20

Open dbarker opened this issue 1 month ago • 2 comments

Fixes #3538

To run clang-tidy on all header files we need to upgrade to version 20.1.0. This version adds the --exclude-header-filter argument enabling analysis of all project header files excluding the api/.../internal/absl headers.

Changes

  • upgrade the clang-tidy CI workflow and devcontainer to clang-tidy version 20 from version 18

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

dbarker avatar Nov 23 '25 22:11 dbarker

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 89.95%. Comparing base (bed305f) to head (be9b574). :warning: Report is 23 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3762      +/-   ##
==========================================
+ Coverage   89.95%   89.95%   +0.01%     
==========================================
  Files         225      225              
  Lines        7158     7163       +5     
==========================================
+ Hits         6438     6443       +5     
  Misses        720      720              

see 5 files with indirect coverage changes

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Nov 23 '25 23:11 codecov[bot]

@dbarker

The generated reports (seen in CI) are very nice.

marcalff avatar Nov 24 '25 23:11 marcalff