search

open-telemetry / opentelemetry-cpp

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

[BUILD] Fix -Wmissing-template-arg-list-after-template-kw warning #3133

Closed yashykt closed 2 weeks ago

yashykt commented 2 weeks ago

Fixes #3124

Changes

Fix for missing-template-arg-list-after-template-kw warning.

netlify[bot] commented 2 weeks ago

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

Name Link
Latest commit e6ee2628901d03f35cafdcc09d02b86a809c0659
Latest deploy log https://app.netlify.com/sites/opentelemetry-cpp-api-docs/deploys/672d3d3873e4c00008b7d630
codecov[bot] commented 2 weeks ago

Codecov Report

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

Project coverage is 87.85%. Comparing base (497eaf4) to head (e6ee262). Report is 158 commits behind head on main.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/open-telemetry/opentelemetry-cpp/pull/3133/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/3133?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 #3133 +/- ## ========================================== + Coverage 87.12% 87.85% +0.73% ========================================== Files 200 195 -5 Lines 6109 6136 +27 ========================================== + Hits 5322 5390 +68 + Misses 787 746 -41 ``` | [Files with missing lines](https://app.codecov.io/gh/open-telemetry/opentelemetry-cpp/pull/3133?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=open-telemetry) | Coverage Δ | | |---|---|---| | [api/include/opentelemetry/logs/event\_logger.h](https://app.codecov.io/gh/open-telemetry/opentelemetry-cpp/pull/3133?src=pr&el=tree&filepath=api%2Finclude%2Fopentelemetry%2Flogs%2Fevent_logger.h&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=open-telemetry#diff-YXBpL2luY2x1ZGUvb3BlbnRlbGVtZXRyeS9sb2dzL2V2ZW50X2xvZ2dlci5o) | `90.00% <100.00%> (-2.30%)` | :arrow_down: | | [api/include/opentelemetry/logs/logger.h](https://app.codecov.io/gh/open-telemetry/opentelemetry-cpp/pull/3133?src=pr&el=tree&filepath=api%2Finclude%2Fopentelemetry%2Flogs%2Flogger.h&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=open-telemetry#diff-YXBpL2luY2x1ZGUvb3BlbnRlbGVtZXRyeS9sb2dzL2xvZ2dlci5o) | `63.05% <100.00%> (-1.53%)` | :arrow_down: | | [...pi/include/opentelemetry/logs/logger\_type\_traits.h](https://app.codecov.io/gh/open-telemetry/opentelemetry-cpp/pull/3133?src=pr&el=tree&filepath=api%2Finclude%2Fopentelemetry%2Flogs%2Flogger_type_traits.h&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=open-telemetry#diff-YXBpL2luY2x1ZGUvb3BlbnRlbGVtZXRyeS9sb2dzL2xvZ2dlcl90eXBlX3RyYWl0cy5o) | `100.00% <100.00%> (+16.00%)` | :arrow_up: | ... and [98 files with indirect coverage changes](https://app.codecov.io/gh/open-telemetry/opentelemetry-cpp/pull/3133/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=open-telemetry)