-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
[chore] docs(builder): remove outdated note on builder version reporting #11261
[chore] docs(builder): remove outdated note on builder version reporting #11261
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #11261 +/- ##
=======================================
Coverage 91.42% 91.42%
=======================================
Files 424 424
Lines 20188 20188
=======================================
Hits 18456 18456
Misses 1348 1348
Partials 384 384 ☔ View full report in Codecov by Sentry. |
Have you tested it? I am confused about #11208 then? |
I think #11208 is a different issue, that PR is about respecting the version number once it is set. This PR is about the (already fixed) incorrect version format. |
@bogdandrutu Yes, I tested it with v0.109.0 and v0.110.0. $ go install go.opentelemetry.io/collector/cmd/[email protected]
$ builder version
ocb version v0.110.0
$ go install go.opentelemetry.io/collector/cmd/[email protected]
$ builder version
ocb version v0.109.0
$ go install go.opentelemetry.io/collector/cmd/builder@latest
$ builder version
ocb version v0.110.0 |
…ing (open-telemetry#11261) This has been fixed since v0.91.0: open-telemetry#8994.
…ing (open-telemetry#11261) This has been fixed since v0.91.0: open-telemetry#8994.
This has been fixed since v0.91.0: #8994.