Skip to content

Conversation

@bidetofevil
Copy link
Contributor

Update baseline README before adding usage example

@bidetofevil bidetofevil requested a review from a team as a code owner November 12, 2025 18:10
@codecov
Copy link

codecov bot commented Nov 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.30%. Comparing base (3907a4e) to head (f646b89).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1385   +/-   ##
=======================================
  Coverage   63.30%   63.30%           
=======================================
  Files         158      158           
  Lines        3109     3109           
  Branches      320      320           
=======================================
  Hits         1968     1968           
  Misses       1046     1046           
  Partials       95       95           

☔ 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.

//...
api(platform("io.opentelemetry.android:opentelemetry-android-bom:0.16.0-alpha"))
implementation("io.opentelemetry.android:android-agent") // Version is resolved thru bom
api(platform("io.opentelemetry.android:opentelemetry-android-bom:<version>"))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have build automation that runs as part of the release process that updates these...so we want to keep the latest specific version here. It will be automatically updated.

Suggested change
api(platform("io.opentelemetry.android:opentelemetry-android-bom:<version>"))
api(platform("io.opentelemetry.android:opentelemetry-android-bom:0.16.0-alpha"))

(This allows folks to easily copy/paste a working example with the latest version).

Copy link
Contributor

@breedx-splk breedx-splk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I had one small fix to merge about the version number, but overall this is a nice change.

Copy link
Contributor

@LikeTheSalad LikeTheSalad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cheers!

@LikeTheSalad LikeTheSalad merged commit 3f632c4 into open-telemetry:main Nov 18, 2025
8 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.

3 participants