i18n: register 10 new locales + check-script portability fixes - #76
Closed
neoOpus wants to merge 4 commits into
Closed
i18n: register 10 new locales + check-script portability fixes#76neoOpus wants to merge 4 commits into
neoOpus wants to merge 4 commits into
Conversation
Register fr, ar, id, tr, ko, uk, nl, fa, bn, hy in the site config (ar/fa RTL), regenerate the url-inventory baseline (650 routes), and fix the check scripts for Windows contributors (fileURLToPath instead of URL.pathname, CRLF-tolerant frontmatter parsing, prebuild path fix). The 10 locale PRs in this set depend on this branch; merge it first.
Author
|
Superseded by PR #87. Infrastructure changes now included there. |
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.
i18n: register 10 new locales + check-script portability fixes
Enables the 10 new locale PRs (
upstream-i18n-fr,-ar,-id,-tr,-ko,-uk,-nl,-fa,-bn,-hy). Merge this first — the localedirectories are ignored by Docusaurus until the locale is registered here.
Merging it alone is safe.
Changes
docusaurus.config.js— registerfr,ar,id,tr,ko,uk,nl,fa,bn,hyini18n.localeswith native labels, textdirection (
rtlforar/fa) andhtmlLang. No other config changes.scripts/url-inventory.txt— baseline regenerated (150 → 650routes) to cover the new locales;
pnpm run checkstays green.src/components/landing/sections.tsx+home.hero.enini18n/{zh-Hans,en,ru}/code.json— the homepage hero tagline("Superpowers for your browser.") was hardcoded English on every locale;
it is now a translatable string (the 10 new locales carry their own
translations in their PRs).
scripts/fetch-landing-stats.mjs—fileURLToPathinstead ofURL.pathnamesopnpm run build's prebuild works on Windows.scripts/check-{frontmatter,i18n-parity,i18n-fallbacks,url-inventory}.mjs— same
fileURLToPathfix, plus CRLF-tolerant frontmatter parsing andbaseline comparison (git checks files out with CRLF on Windows).
Verified
pnpm run build— passes with all 13 locales.pnpm run check— URL inventory (650 routes), fallbacks, strict i18nparity (10 new locales: 44 docs each, 0 fallback) and strict
frontmatter (no violations) all pass on the merged tree.
RTL for
ar) and a French Agent API page.Deliberately not included
that would alter upstream's look or reintroduce assets upstream dropped.
Available as a separate follow-up if desired.
de,es,it,ja,pt,vi,zh-Hant) —incomplete on the fork; not ready for upstream yet.