Skip to content

Conversation

@svetlanabrennan
Copy link
Contributor

In my review of the api-logs and sdk-logs for browser friendliness in this issue, I found a few imports that we can consolidate.

@svetlanabrennan svetlanabrennan requested a review from a team as a code owner July 14, 2025 21:50
@svetlanabrennan svetlanabrennan added the sdk:logs Issues and PRs related to the Logs SDK label Jul 14, 2025
@svetlanabrennan svetlanabrennan added this to the Logs API/SDK GA milestone Jul 14, 2025
@codecov
Copy link

codecov bot commented Jul 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.01%. Comparing base (cb42f7d) to head (0bac8bb).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5801      +/-   ##
==========================================
- Coverage   95.01%   95.01%   -0.01%     
==========================================
  Files         303      303              
  Lines        7946     7943       -3     
  Branches     1607     1607              
==========================================
- Hits         7550     7547       -3     
  Misses        396      396              
Files with missing lines Coverage Δ
...xperimental/packages/sdk-logs/src/LogRecordImpl.ts 98.26% <100.00%> (-0.02%) ⬇️
...sdk-logs/src/export/BatchLogRecordProcessorBase.ts 93.54% <ø> (-0.07%) ⬇️
...es/sdk-logs/src/export/ConsoleLogRecordExporter.ts 83.33% <100.00%> (-1.29%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@svetlanabrennan svetlanabrennan added this pull request to the merge queue Jul 23, 2025
Merged via the queue into open-telemetry:main with commit 48538b4 Jul 23, 2025
25 checks passed
@svetlanabrennan svetlanabrennan deleted the audit-browser-friendliness branch July 23, 2025 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sdk:logs Issues and PRs related to the Logs SDK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants