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

Update dependency versions. #304

Merged

Conversation

breedx-splk
Copy link
Contributor

The instrumentation bom is not alpha, so this updates that while updating us to the latest. It also updates the gradle version wrapper to the latest name.

@breedx-splk breedx-splk requested a review from a team as a code owner April 15, 2024 21:49
@@ -1,6 +1,7 @@
[versions]
opentelemetry = "1.37.0"
opentelemetry-alpha = "2.2.0-alpha"
opentelemetry-inst = "2.3.0"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's clearer with inst in the name, to know when you have the instrumentation version (vs. core version).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the idea of replacing alpha with something more meaningful. I would prefer to avoid abbreviations though, I know opentelemetry-instrumentation could be a lot, though the first time I saw inst, for some reason "instrumentation" wasn't the first word that came to my mind 😅

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, fine, I'll spell out inst -> instrumentation...but I'm not going to change semconv. :)

@breedx-splk breedx-splk merged commit 6c16386 into open-telemetry:main Apr 23, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants