-
Notifications
You must be signed in to change notification settings - Fork 417
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CI] Upgrade to clang-format 18 #2684
[CI] Upgrade to clang-format 18 #2684
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2684 +/- ##
==========================================
+ Coverage 87.12% 87.68% +0.56%
==========================================
Files 200 190 -10
Lines 6109 5849 -260
==========================================
- Hits 5322 5128 -194
+ Misses 787 721 -66
|
We can also update Dockerfile to use the appropriate docker image so that: ./ci/run_docker.sh ./ci/do_ci.sh format use the correct version. |
I assume this comes from the infrastructure, not from the opentelemetry-cpp codebase. |
Weird it's only coming for maintainer mode CI checks. |
Fixes #2578
Changes
Please provide a brief description of the changes here.
CI
to use clang-format-18 on ubuntu-24-04For significant contributions please make sure you have completed the following items:
CHANGELOG.md
updated for non-trivial changes