-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Java API - Update link for specs reference #8287
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
Java API - Update link for specs reference #8287
Conversation
chalin
left a comment
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.
👌
|
Editorial change, so I'll merge. |
Signed-off-by: Vitor Vasconcellos <[email protected]>
Signed-off-by: Vitor Vasconcellos <[email protected]>
dd4589f to
cf701cc
Compare
chalin
left a comment
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.
Rebased. LGTM if all checks pass.
|
@vitorvasc - oops, did you miss one? See the link-check failure in https://github.com/open-telemetry/opentelemetry.io/actions/runs/18948532019/job/54106781326?pr=8183
|
Fixes the broken anchor link in Portuguese Java API docs from `#standard-attribute` to `#attribute` to match the spec changes. This follows the same fix applied in #8287 (English and Japanese) and #8291 (Chinese) but was missed for the Portuguese translation. Co-authored-by: chalin <[email protected]>
As the latest (unreleased) version of the specification is currently being tracked in #8183, we noticed a broken link in the Java API page. The link currently refers to a section that will be removed in the next specification update.
This PR updates that link to the correct location and fixes the issue.
Preview: https://deploy-preview-8287--opentelemetry.netlify.app/docs/languages/java/api/#attributes