open-telemetry / opentelemetry-cpp

The OpenTelemetry C++ Client
https://opentelemetry.io/
Apache License 2.0
890 stars 426 forks source link

[CI] Add cppcheck in the build #3151

Closed chusitoo closed 1 week ago

chusitoo commented 1 week ago

Fixes #2297

Changes

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

netlify[bot] commented 1 week ago

Deploy Preview for opentelemetry-cpp-api-docs canceled.

Name Link
Latest commit a19d070897e81907dddc4158c83968d965ccd8dc
Latest deploy log https://app.netlify.com/sites/opentelemetry-cpp-api-docs/deploys/673bd648072f580008f43277
codecov[bot] commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.83%. Comparing base (1695253) to head (a19d070). Report is 1 commits behind head on main.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/open-telemetry/opentelemetry-cpp/pull/3151/graphs/tree.svg?width=650&height=150&src=pr&token=FJESTYQ2AD&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=open-telemetry)](https://app.codecov.io/gh/open-telemetry/opentelemetry-cpp/pull/3151?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=open-telemetry) ```diff @@ Coverage Diff @@ ## main #3151 +/- ## ======================================= Coverage 87.83% 87.83% ======================================= Files 195 195 Lines 6146 6146 ======================================= Hits 5398 5398 Misses 748 748 ```

🚨 Try these New Features:

ThomsonTan commented 1 week ago

Is this a subset of the part 2 change?

marcalff commented 1 week ago

Is this a subset of the part 2 change?

Yes.

Once part 1 is merged to main, part 2 PR will only contain part 2 (the code fixes).