Skip to content

docs: fix broken relative links in translated READMEs - #3858

Open
ZayanKhan-12 wants to merge 1 commit into
sharkdp:masterfrom
ZayanKhan-12:fix-translated-readme-links
Open

docs: fix broken relative links in translated READMEs#3858
ZayanKhan-12 wants to merge 1 commit into
sharkdp:masterfrom
ZayanKhan-12:fix-translated-readme-links

Conversation

@ZayanKhan-12

Copy link
Copy Markdown

The translated READMEs (doc/README-ja.md, -ko.md, -ru.md, -zh.md) live one directory deeper than the English README.md, but several links were copied over without adjusting for that, so they 404 when viewed on GitHub:

  • Manpage syntax (ja, ko, ru, zh) — dropped the 02_Extra/ path component and/or the ../ prefix; the file is at assets/syntaxes/02_Extra/Manpage.sublime-syntax
  • doc/alternatives.md / doc/assets.md (ru, zh) — resolved to doc/doc/...; corrected to alternatives.md / assets.md
  • CONTRIBUTING.md, LICENSE-APACHE, LICENSE-MIT (ru, zh) — pointed inside doc/ instead of the repository root; corrected to ../…

The translations were already inconsistent (ko had ../ but the wrong 02_Extra; ru/zh had 02_Extra but no ../), which is what surfaced this.

Verification

Resolved every relative link in all doc/README-*.md against the filesystem before and after: the four files had 12 broken links; after the change 0 remain, and no other translated README is affected. Docs-only.

🤖 Generated with Claude Code

The Japanese, Korean, Russian and Chinese READMEs live in doc/, but
several links were copied from the root README without adjusting for
the extra directory level, so they 404 on GitHub:

- the Manpage syntax link dropped the 02_Extra/ path component and/or
  the ../ prefix (ja, ko, ru, zh)
- doc/alternatives.md and doc/assets.md resolved to doc/doc/... (ru, zh)
- CONTRIBUTING.md, LICENSE-APACHE and LICENSE-MIT pointed inside doc/
  instead of the repository root (ru, zh)

All links now resolve to existing files; verified by resolving every
relative link in the translated READMEs against the filesystem.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.

2 participants