Skip to content

fix: break circular dev-dep between appender-tracing and stdout#3509

Merged
cijothomas merged 1 commit into
open-telemetry:mainfrom
cijothomas:cijothomas/fix-circular-devdep
May 11, 2026
Merged

fix: break circular dev-dep between appender-tracing and stdout#3509
cijothomas merged 1 commit into
open-telemetry:mainfrom
cijothomas:cijothomas/fix-circular-devdep

Conversation

@cijothomas

Copy link
Copy Markdown
Member

Remove opentelemetry-stdout dev-dependency from opentelemetry-appender-tracing to break the circular dev-dep that prevents cargo publish (Cargo now validates dev-deps against the crates.io index during packaging).

Doc examples and the integration test use InMemoryLogExporter instead. The runnable example is already in examples/logs-basic/. Also fixes publish order in publish.sh.

@cijothomas cijothomas requested a review from a team as a code owner May 9, 2026 00:41
@codecov

codecov Bot commented May 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.8%. Comparing base (ec289cb) to head (43c87dc).

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #3509   +/-   ##
=====================================
  Coverage   82.8%   82.8%           
=====================================
  Files        130     130           
  Lines      27289   27289           
=====================================
  Hits       22622   22622           
  Misses      4667    4667           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cijothomas cijothomas added this pull request to the merge queue May 11, 2026
Merged via the queue into open-telemetry:main with commit dfdc478 May 11, 2026
26 checks passed
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.

2 participants