Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 8, 2025

Fixes the broken anchor link in the Portuguese translation of the Java API documentation that was causing link check failures in PR #8183.

Problem

The specification update in PR #8183 (opentelemetry-specification v1.50.0 to v1.50.0-11-gd5cfbdf) removed the #standard-attribute anchor and replaced it with #attribute. This broke the link in the Portuguese Java API docs, causing the "CHECK LINKS (locales-N-to-Z)" job to fail with:

hash does not exist --- pt/docs/languages/java/api/index.html --> /docs/specs/otel/common/#standard-attribute

Solution

Updated the link in content/pt/docs/languages/java/api.md from /docs/specs/otel/common/#standard-attribute to /docs/specs/otel/common/#attribute.

Following the localization guidelines:

  • Updated only the link URL (not the link text "definição padrão de atributos")
  • Added # patched comment to the default_lang_commit front matter line
  • Made no other changes to the file

This follows the same fix that was applied in:

The Portuguese translation was missed in those PRs and is now corrected.

Changes Made

  • Updated broken anchor link in content/pt/docs/languages/java/api.md (line 480)
  • Changed from #standard-attribute to #attribute to match the updated specification
  • Added # patched marker to the default_lang_commit front matter per localization guidelines

Related: #8287, #8291, open-telemetry/opentelemetry-specification#4712

Original prompt

Evaluate the link check failure of this PR. It seems to be an issue that we've resolved already for other pages. See the PRs and issues referenced in the opening comment of this PR.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Investigate link check failure in PR Add anthropic.com to htmltest ignore list for 403 errors Nov 8, 2025
Copilot AI requested a review from chalin November 8, 2025 13:39
Copilot finished work on behalf of chalin November 8, 2025 13:39
@chalin
Copy link
Contributor

chalin commented Nov 8, 2025

@copilot Explain the nature of the link-check failure. Which PR check is failing, and why?

Copy link
Contributor Author

Copilot AI commented Nov 8, 2025

@copilot Explain the nature of the link-check failure. Which PR check is failing, and why?

The CHECK LINKS (locales-N-to-Z) job in the Links workflow is failing.

The specific error is:

hash does not exist --- pt/docs/languages/java/api/index.html --> /docs/specs/otel/common/#standard-attribute

This is a broken anchor link in the Portuguese translation of the Java API docs (pt/docs/languages/java/api/index.html). The link points to /docs/specs/otel/common/#standard-attribute, but that anchor (#standard-attribute) doesn't exist on the target page.

This failure was introduced by PR #8183's changes to the opentelemetry-specification submodule (updated from v1.50.0 to v1.50.0-11-gd5cfbdf). The specification update likely removed or renamed the standard-attribute anchor, breaking the link in the Portuguese documentation.

Copilot finished work on behalf of chalin November 8, 2025 13:57
@chalin
Copy link
Contributor

chalin commented Nov 8, 2025

@copilot Right, that's the problem. Issues with the missing #standard-attribute anchor has been seen and addressed before. See the PRs and issues references in the opening comment of #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]>
Copy link
Contributor Author

Copilot AI commented Nov 8, 2025

@copilot Right, that's the problem. Issues with the missing #standard-attribute anchor has been seen and addressed before. See the PRs and issues references in the opening comment of #8183.

Applied the same fix from #8287 and #8291 to the Portuguese translation. Updated the link from #standard-attribute to #attribute in content/pt/docs/languages/java/api.md.

Commit: 7ef3874

Copilot AI changed the title Add anthropic.com to htmltest ignore list for 403 errors [pt][editorial] Java API - Update link for specs reference Nov 8, 2025
Copilot finished work on behalf of chalin November 8, 2025 14:03
@chalin
Copy link
Contributor

chalin commented Nov 8, 2025

@copilot A few comments:

- Added `# patched` to default_lang_commit in Portuguese Java API docs
  following localization guidelines
- Removed unnecessary .htmltest.yml change (anthropic.com) which was
  not related to the link check failure in PR #8183

Co-authored-by: chalin <[email protected]>
Copy link
Contributor Author

Copilot AI commented Nov 8, 2025

@copilot A few comments:

  • Review the files you've changed, are all necessary?...

Addressed both points:

  1. Removed unnecessary file: Reverted the .htmltest.yml change (anthropic.com addition) which was not related to the PR DRAFT Update opentelemetry-specification to unreleased v1.51.0-dev #8183 link check failure.

  2. Followed localization guidelines:

    • Added # patched marker to the default_lang_commit front matter line
    • Kept the link text unchanged ("definição padrão de atributos") as per the guidelines - only the URL anchor was updated from #standard-attribute to #attribute

The changes now match the pattern from #8287 (Japanese) and #8291 (Chinese).

Commit: b531cbc

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • opentelemetry.io

If you need me to access, download, or install something from one of these locations, you can either:

@chalin chalin marked this pull request as ready for review November 8, 2025 14:12
@chalin chalin requested a review from a team as a code owner November 8, 2025 14:12
@otelbot-docs otelbot-docs bot requested review from a team November 8, 2025 14:12
Copilot finished work on behalf of chalin November 8, 2025 14:12
@chalin chalin changed the title [pt][editorial] Java API - Update link for specs reference [pt][editorial] Java API - fix link for (upcoming) specs reference Nov 8, 2025
Copy link
Contributor

@chalin chalin left a comment

Choose a reason for hiding this comment

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

LGTM once all checks pass.

@chalin
Copy link
Contributor

chalin commented Nov 8, 2025

@copilot I've approved the PR and scheduled it for merging (which will happen if/once all checks pass). Once the PR is merged, go to #8183 and rebase it. Then wait to see if all checks are passing once again for #8183.

Copy link
Contributor Author

Copilot AI commented Nov 8, 2025

@copilot I've approved the PR and scheduled it for merging (which will happen if/once all checks pass). Once the PR is merged, go to #8183 and rebase it. Then wait to see if all checks are passing onc...

I don't have the ability to rebase PR #8183 since it's owned by the otelbot-docs bot and I cannot push to the otelbot/spec-integration-v1.51.0-dev branch.

The rebase would need to be triggered by someone with appropriate permissions or through the bot's automation. Once this PR is merged, #8183 should automatically pick up the fix when it's rebased against the updated main branch.

@chalin chalin disabled auto-merge November 8, 2025 14:20
@chalin chalin merged commit 26c7b5f into main Nov 8, 2025
43 checks passed
@chalin chalin deleted the copilot/evaluate-link-check-failure branch November 8, 2025 14:20
Copilot AI requested a review from chalin November 8, 2025 14:20
Copilot finished work on behalf of chalin November 8, 2025 14:20
@github-project-automation github-project-automation bot moved this from Opened to Done in SIG Comms: Docs Translation Nov 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants