-
Notifications
You must be signed in to change notification settings - Fork 534
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: update experimental Otel deps to ^0.31.0 #1096
Conversation
There seems to be some dependency issue in opentelemetry-resource-detector-github resulting in failing builds:
@smithclay I think you added this detector but you are not mentioned as owner in https://github.com/open-telemetry/opentelemetry-js-contrib/blob/main/.github/component_owners.yml. |
Codecov Report
@@ Coverage Diff @@
## main #1096 +/- ##
==========================================
- Coverage 96.07% 92.00% -4.08%
==========================================
Files 14 121 +107
Lines 892 5978 +5086
Branches 191 1171 +980
==========================================
+ Hits 857 5500 +4643
- Misses 35 478 +443
|
back to draft as 0.30.0 is already outdated |
d9d3043
to
b59e26c
Compare
Depend on ^0.31.0 for experimental OTel packages because ^0.29.2 doesn't pick up 0.31.x. This avoids duplicates of e.g. @opentelemetry/instrumentation in case latest is requested via app package.json.
b59e26c
to
ebcf0e3
Compare
I'm going to hold this until I can release the packages that have already been updated. #1112 is blocking that so please review that so we can get the previous release finished and we can get this through |
Going to merge this but i'd like to do #1113 first since this touches so many packages. Would appreciate a review there. |
Which problem is this PR solving?
Avoid duplicates of e.g.
@opentelemetry/instrumentation
.Short description of the changes
Depend on ^0.31.0 for experimental OTel packages because ^0.29.2 doesn't pick up 0.31.
This avoids duplicates of e.g. @opentelemetry/instrumentation in case latest is requested via
app package.json.
Checklist
npm run test-all-versions
for the edited package(s) on the latest commit if applicable.