-
Notifications
You must be signed in to change notification settings - Fork 701
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
opentelemetry: rename Layer to Subscriber #1226
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thanks @Folyd
While this looks good to me, I'm guessing that this shouldn't get backported to the |
@davidbarsky yeah this shouldn't get backported |
@davidbarsky looks like this pr is just waiting on your 👍 / 👎 |
cd1050d
to
0ba17b0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry! This looks good to me. Approved!
I think we can get this PR merged? |
Looks like I don't have permission to merge this one, @davidbarsky or @hawkw? |
Done! |
This PR renames the following:
tracing_opetelemetry::layer
totracing_opetelemetry::subscriber
tracing_opetelemetry::layer::layer()
totracing_opetelemetry::subscriber::subscriber()
tracing_opetelemetry::layer::OpenTelemetryLayer
totracing_opetelemetry::subscriber::OpenTelemetrySubscriber
Related PR: #1015