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.

Sign up for GitHub

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

Jump to bottom

Do not register application tracer #8

Closed

rlopzc wants to merge 1 commit into salemove:master from rlopzc:patch-1

rlopzc commented Nov 8, 2023

There is no need to register the application tracer.

OpenTelemetry automatically registers the application tracer. They removed the function on the version 1.0.0.0-rc.3. Here is the changelog.


          Do not register application tracer

3c811f8

There is no need to register the application tracer.

OpenTelemetry automatically registers the application tracer. They removed the function on the version `1.0.0.0-rc.3`. [Here](https://github.com/open-telemetry/opentelemetry-erlang/blob/0ec0e863215466dbdec153b99b2873e1e695ce22/CHANGELOG.md?plain=1#L468) is the changelog.

indrekj approved these changes

View reviewed changes

Author

rlopzc commented Nov 9, 2023

Hello @indrekj, the PR can't be merged. It says that a review is required but you already approved the PR 🤷‍♂️ haha

rlopzc closed this by deleting the head repository

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet