opentelemetry-java icon indicating copy to clipboard operation
opentelemetry-java copied to clipboard

Include the incubating dependency in the published pom as an optional dependency

Open trask opened this issue 10 months ago • 2 comments

Resolves #7119

The published poms for the related modules, e.g. for io.opentelemetry:opentelemetry-sdk-trace, would now include:

    <dependency>
      <groupId>io.opentelemetry</groupId>
      <artifactId>opentelemetry-api-incubator</artifactId>
      <version>1.48.0-alpha</version>
      <scope>runtime</scope>
      <optional>true</optional>
    </dependency>
  </dependencies>

trask avatar Feb 22 '25 19:02 trask

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 89.85%. Comparing base (56941a5) to head (d1ba7c4). :warning: Report is 346 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #7137   +/-   ##
=========================================
  Coverage     89.85%   89.85%           
  Complexity     6610     6610           
=========================================
  Files           740      740           
  Lines         19986    19986           
  Branches       1964     1964           
=========================================
  Hits          17958    17958           
  Misses         1439     1439           
  Partials        589      589           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

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

codecov[bot] avatar Feb 22 '25 19:02 codecov[bot]

See https://github.com/open-telemetry/opentelemetry-java/pull/7157 for a possible alternative

jack-berg avatar Feb 27 '25 21:02 jack-berg

what's the status on this, @trask ? Let's get it merged or closed in favor of the other one.

jkwatson avatar Sep 11 '25 16:09 jkwatson

closing in favor of the less invasive #7157

trask avatar Sep 11 '25 17:09 trask