Skip to content

security/tls: the translated heading keeps its English anchor - #37

Merged
tannevaled merged 1 commit into
mainfrom
keep-tls-anchor
Oct 11, 2026
Merged

tannevaled merged 1 commit into
mainfrom
keep-tls-anchor

Conversation

@tannevaled

Copy link
Copy Markdown
Contributor

The heading "When the certificate is asked for" (security/tls.md) was translated in fr, es and de without {#when-the-certificate-is-asked-for}. So a link to that fragment lands at the top of the page in three languages. That is exactly what the README's rule is there to prevent: a translated heading keeps the English anchor as an explicit id. No link targets it today; the next one would have failed silently.

This adds the explicit id in the three languages, which takes one line each.

Found by go-fleettools' new i18ncheck (go-fleettools/fleettools#73). With this change it reads the tree with 0 problems, using -comments, because the configuration examples' comments are translated on purpose. Hugo builds with --panicOnWarning, and the anchor is present in the built fr, es and de pages.

🤖 Generated with Claude Code

"When the certificate is asked for" was translated in fr, es and de
without {#when-the-certificate-is-asked-for}, so a link to that fragment
would land at the top of the page in three languages -- what the README's
rule (a translated heading keeps the English anchor as an explicit id) is
there to prevent. No link targets it today; the next one would have
failed silently.

Found by go-fleettools' new i18ncheck, which now reads this tree with 0
problems (with -comments: the configuration examples' comments are
translated on purpose). Hugo builds with --panicOnWarning; the anchor is in
the built fr, es and de pages.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@tannevaled
tannevaled merged commit 45ef6ec into main Oct 11, 2026
3 checks passed
@tannevaled
tannevaled deleted the keep-tls-anchor branch October 11, 2026 16:34
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.

1 participant