Skip to content

Added anchors to internal-telemetry.md#8325

Merged
chalin merged 1 commit intoopen-telemetry:mainfrom
Andygol:internal-telemetry-anchors
Nov 4, 2025
Merged

Added anchors to internal-telemetry.md#8325
chalin merged 1 commit intoopen-telemetry:mainfrom
Andygol:internal-telemetry-anchors

Conversation

@Andygol
Copy link
Copy Markdown
Contributor

@Andygol Andygol commented Nov 3, 2025

This PR added explicit anchors to avoid Prettier replacing _ --> \_ in the anchors on localized pages.

@Andygol Andygol requested a review from a team as a code owner November 3, 2025 22:10
@otelbot-docs otelbot-docs bot requested review from a team and mx-psi and removed request for a team November 3, 2025 22:10
Copy link
Copy Markdown
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.

I confirm that the _ causes problems when the heading ID is given explicitly. Thanks for this fix @Andygol.

@chalin
Copy link
Copy Markdown
Contributor

chalin commented Nov 4, 2025

Editorial. Merging.

@chalin chalin merged commit 4b43163 into open-telemetry:main Nov 4, 2025
22 checks passed
@otelbot
Copy link
Copy Markdown
Contributor

otelbot bot commented Nov 4, 2025

Thank you for your contribution @Andygol! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey.

@Andygol Andygol deleted the internal-telemetry-anchors branch November 4, 2025 08:01
dmitchsplunk pushed a commit to dmitchsplunk/opentelemetry.io that referenced this pull request Nov 7, 2025
jaehanbyun pushed a commit to jaehanbyun/opentelemetry.io that referenced this pull request Dec 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants