Skip to content
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

Merge opentelemetry-api-events into opentelemetry-extension-incubator #6121

Closed
jack-berg opened this issue Jan 5, 2024 · 3 comments · Fixed by #6289
Closed

Merge opentelemetry-api-events into opentelemetry-extension-incubator #6121

jack-berg opened this issue Jan 5, 2024 · 3 comments · Fixed by #6289

Comments

@jack-berg
Copy link
Member

jack-berg commented Jan 5, 2024

Writing this made me realize that its silly that we incubate APIs in both opentelemetry-api-events and opentelemetry-extension-incubator. I propose we merge events into opentelemetry-extension-incubator, and do it in lockstep with #6001 to minimize churn.

Originally posted by @jack-berg in #6120 (comment)

@trask
Copy link
Member

trask commented Jan 5, 2024

adding #6120 (comment) here as well:

maybe opentelemetry-extension-incubator -> opentelemetry-api-incubator?

since it's not really an incubator for extensions

(also https://github.com/open-telemetry/opentelemetry-java-instrumentation/tree/main/instrumentation-api-incubator)

@jack-berg
Copy link
Member Author

maybe opentelemetry-extension-incubator -> opentelemetry-api-incubator?

Yeah that makes sense to me.

@jkwatson
Copy link
Contributor

jkwatson commented Jan 5, 2024

seems fine to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants