Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[BUILD] Adding typecast without whom c++latest build fails #3281

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

Merged

Conversation

jeremicmilan
Copy link
Contributor

Fixes #3237

Changes

Adding typecast without whom c++latest build fails. More details in #3237.

@jeremicmilan jeremicmilan requested a review from a team as a code owner February 19, 2025 10:50
Copy link

linux-foundation-easycla bot commented Feb 19, 2025

CLA Signed


The committers listed above are authorized under a signed CLA.

Copy link

netlify bot commented Feb 19, 2025

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

Name Link
🔨 Latest commit eb3d0ad
🔍 Latest deploy log https://app.netlify.com/sites/opentelemetry-cpp-api-docs/deploys/67b5b7945c8b5d0009d29bf1

@jeremicmilan jeremicmilan changed the title Adding typecast without whom c++26 build fails Adding typecast without whom c++latest build fails Feb 19, 2025
Copy link

codecov bot commented Feb 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.96%. Comparing base (821ad9e) to head (eb3d0ad).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #3281   +/-   ##
=======================================
  Coverage   87.96%   87.96%           
=======================================
  Files         205      205           
  Lines        6407     6407           
=======================================
  Hits         5635     5635           
  Misses        772      772           

@ThomsonTan
Copy link
Contributor

@jeremicmilan could you please sign the CLA then we could get this merged.

@jeremicmilan
Copy link
Contributor Author

@ThomsonTan, I did. It's waiting on approval.

@jeremicmilan
Copy link
Contributor Author

Ok, I just signed with my personal account. I did not read through carefully the first time around.

@lalitb lalitb merged commit b89e63c into open-telemetry:main Feb 20, 2025
57 checks passed
malkia added a commit to malkia/opentelemetry-cpp that referenced this pull request Feb 20, 2025
Adding typecast without whom c++26 build fails (open-telemetry#3281)
@marcalff marcalff changed the title Adding typecast without whom c++latest build fails [BUILD] Adding typecast without whom c++latest build fails Mar 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compiling etw_logger_exporter.h with c++latest fails
5 participants