Skip to content

Conversation

@zurex
Copy link
Contributor

@zurex zurex commented Apr 20, 2025

This PR one of a set PRs that used to address #3370 .

Changes

Fix the documentation comments of logger.h under API

@zurex zurex requested a review from a team as a code owner April 20, 2025 08:29
@netlify
Copy link

netlify bot commented Apr 20, 2025

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

Name Link
🔨 Latest commit ac0ef69
🔍 Latest deploy log https://app.netlify.com/sites/opentelemetry-cpp-api-docs/deploys/6804b066f4ef98000847b1da

@codecov
Copy link

codecov bot commented Apr 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.56%. Comparing base (f987c9c) to head (ac0ef69).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #3371   +/-   ##
=======================================
  Coverage   89.56%   89.56%           
=======================================
  Files         210      210           
  Lines        6502     6502           
=======================================
  Hits         5823     5823           
  Misses        679      679           
Files with missing lines Coverage Δ
api/include/opentelemetry/logs/logger.h 63.83% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@psx95 psx95 changed the title Fix the documention comments of logger.h under API [DOC] Fix documentation tags in logger.h Apr 21, 2025
@psx95 psx95 changed the title [DOC] Fix documentation tags in logger.h [DOC] Fix documentation tags in logger API Apr 21, 2025
@lalitb lalitb merged commit acd4a0e into open-telemetry:main Apr 21, 2025
66 checks passed
malkia added a commit to malkia/opentelemetry-cpp that referenced this pull request Apr 21, 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.

3 participants