fix: use absolute paths for calendar and blog links, add htmltest config#2088
Open
TerryHowe wants to merge 2 commits into
Open
fix: use absolute paths for calendar and blog links, add htmltest config#2088TerryHowe wants to merge 2 commits into
TerryHowe wants to merge 2 commits into
Conversation
The calendar/release and blog/helm-dco hrefs were relative, causing broken links on all non-English locale pages (e.g. /ja/calendar/release has no Netlify redirect, only /calendar/release does). Also add .htmltest.yml to configure htmltest for this project: - IgnoreDirectoryMissingTrailingSlash: suppresses false positives from Docusaurus building directory-based URLs (Netlify handles trailing slash redirects natively) - CheckInternalHash: false: skips empty hash (#) nav toggle links; Docusaurus already validates real anchor links via onBrokenLinks - IgnoreURLs: skips locale-specific 404 pages that Docusaurus does not generate (only one global 404.html is built) Signed-off-by: Terry Howe <terrylhowe@gmail.com>
…an locale
Add Docusaurus {#anchor-id} explicit IDs to Russian translation headings so
that links using English anchor names resolve correctly:
- release_policy.md: Мажорные релизы → {#major-releases}
- charts.md: Файл Chart.yaml → {#the-chartyaml-file}
- charts.md: Файлы Schema → {#schema-files}
- advanced.md: Бэкенды хранения → {#storage-backends}
- quickstart.md: Инициализация репозитория чартов Helm → {#initialize-a-helm-chart-repository}
- examples.mdx: Драйвер → {#driver}
- charts_hooks.md: Политики удаления хуков → {#hook-deletion-policies}
- chart_repository.md: Сервер репозитория ChartMuseum → {#chartmuseum-repository-server}
Fixes Docusaurus broken anchor warnings during build.
Signed-off-by: Terry Howe <terrylhowe@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
href="calendar/release"→href="/calendar/release"inHomeCommunitycomponent — the relative path breaks on all non-English locale pages (e.g. from/ja/, the link resolves to/ja/calendar/releasewhich has no Netlify redirect; only/calendar/releaseis redirected to Google Calendar)href="blog/helm-dco"→href="/blog/helm-dco"— same relative path issue across all i18n locale homepages.htmltest.ymlto configurebin/htmltestfor this project, eliminating ~383K false-positive errors fromhtmltestwhile keeping meaningful checks:IgnoreDirectoryMissingTrailingSlash: true— Docusaurus builds directory-based URLs; Netlify handles trailing-slash redirects natively, so these are not real broken linksCheckInternalHash: false— suppresses false positives fromhref="#"mobile nav toggles; Docusaurus already validates real anchor links at build time viaonBrokenLinks: "throw"IgnoreURLs: ["^(/[a-z]+)?/404$"]— Docusaurus builds only one global404.html; locale-specific 404 pages (/de/404,/ja/404, etc.) in the language switcher do not exist as files but are handled at the CDN/server levelTest plan
bin/htmltest --skip-externalpasses with 0 errors after rebuilding (the two link fixes require a rebuild to reflect in the built HTML)/calendar/releasestill redirects to Google Calendar on the deployed preview/blog/helm-dcopage loads correctly from all locale homepages🤖 Generated with Claude Code