feat(otel): Add initial package for otel#6002
Conversation
87aa39f to
5cd1925
Compare
Lms24
left a comment
There was a problem hiding this comment.
LGTM, I like the idea of having this in its own package. Just had a two remarks/questions.
Another question: Do you see any problems with the approach of the Otel package requiring sentry/tracing? I'm thinking for example about our long-term plan of splitting up or integrating the tracing package into browser/node so that users only have to install one package anymore.
Discussed this offline - the answer is: yes it might have consequences, but for now not going to worry about it until v8. |
size-limit report 📦
|
ref: #6000
Add boilerplate package for OpenTelemetry Node.
I would like to make this a separate package because we are dependent on the https://github.com/open-telemetry/opentelemetry-js-api, and I don't want to include that as a dependency for everyone else.
The nice thing is that we can require the usage of
@sentry/tracing, so users only have to install@sentry/opentelemetry-node! Aka either use@sentry/tracingOR use@sentry/opentelemetry-node.As such, installation looks like so: