docs(i18n): add Spanish translations (README, CONTRIBUTING, SECURITY) - #799
Open
kmikodev wants to merge 2 commits into
Open
docs(i18n): add Spanish translations (README, CONTRIBUTING, SECURITY)#799kmikodev wants to merge 2 commits into
kmikodev wants to merge 2 commits into
Conversation
Adds Spanish versions of the top-level docs as separate *.es.md files, with a reciprocal language switcher linking each English doc to its Spanish counterpart (and back). - README.es.md, CONTRIBUTING.es.md, SECURITY.es.md: full translations. Code blocks, commands, paths, CLI flags, brand/model names, badges and table structure preserved; only prose translated. README internal anchors point to the translated headings. - README.md / CONTRIBUTING.md / SECURITY.md: add an "English · Español" switcher line near the top. - Drive-by fix: the "Additional Resources" link to the auto-updater doc pointed to a non-existent docs/AUTOUPDATER_QUICKSTART.md; repointed to the actual docs/content/docs/developer/autoupdater.mdx in both EN and ES. Verified: all internal anchors and local file links resolve in all six files. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughSe añaden versiones en español de README, CONTRIBUTING y SECURITY, junto con enlaces de idioma entre las páginas en inglés y español. También se amplían secciones de uso, contribución, desarrollo y política de seguridad. ChangesDocumentación bilingüe y políticas
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 markdownlint-cli2 (0.22.1)README.es.mdmarkdownlint-cli2 wrapper config was not available before execution Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@README.es.md`:
- Around line 318-323: The README markdown examples are missing fence language
labels, leaving markdownlint MD040 active. Update the fenced blocks in the
sections containing the Claude MCP command and the directory tree to use
appropriate language tags, and make sure both affected examples are labeled
consistently so the lint warning is cleared.
- Around line 104-108: Remove the blank lines inside the blockquote in
README.es.md so the callout stays as consecutive quoted lines and satisfies
markdownlint MD028. Keep the “Ver todos los binarios”, “Linux”, and
“¿Problemas?” lines directly adjacent with no empty quoted lines between them,
preserving the existing links and text.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 3a78bc65-3e15-4a9d-9734-07ab7f4c604e
📒 Files selected for processing (6)
CONTRIBUTING.es.mdCONTRIBUTING.mdREADME.es.mdREADME.mdSECURITY.es.mdSECURITY.md
Comment on lines
+104
to
+108
| > **[Ver todos los binarios →](https://github.com/jamiepine/voicebox/releases/latest)** | ||
|
|
||
| > **Linux** — Aún no hay binarios precompilados. Consulta [voicebox.sh/linux-install](https://voicebox.sh/linux-install) para las instrucciones de compilación desde el código fuente. | ||
|
|
||
| > **¿Problemas?** Consulta la [guía de solución de problemas](docs/content/docs/overview/troubleshooting.mdx) para incidencias comunes de instalación, generación, descarga de modelos y GPU. |
Contributor
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Remove the blank lines inside this blockquote.
Markdownlint is already flagging MD028 here. Keeping these callouts on consecutive > lines will keep the doc lint-clean.
♻️ Proposed fix
> **[Ver todos los binarios →](https://github.com/jamiepine/voicebox/releases/latest)**
-
> **Linux** — Aún no hay binarios precompilados. Consulta [voicebox.sh/linux-install](https://voicebox.sh/linux-install) para las instrucciones de compilación desde el código fuente.
-
> **¿Problemas?** Consulta la [guía de solución de problemas](docs/content/docs/overview/troubleshooting.mdx) para incidencias comunes de instalación, generación, descarga de modelos y GPU.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| > **[Ver todos los binarios →](https://github.com/jamiepine/voicebox/releases/latest)** | |
| > **Linux** — Aún no hay binarios precompilados. Consulta [voicebox.sh/linux-install](https://voicebox.sh/linux-install) para las instrucciones de compilación desde el código fuente. | |
| > **¿Problemas?** Consulta la [guía de solución de problemas](docs/content/docs/overview/troubleshooting.mdx) para incidencias comunes de instalación, generación, descarga de modelos y GPU. | |
| > **[Ver todos los binarios →](https://github.com/jamiepine/voicebox/releases/latest)** | |
| > **Linux** — Aún no hay binarios precompilados. Consulta [voicebox.sh/linux-install](https://voicebox.sh/linux-install) para las instrucciones de compilación desde el código fuente. | |
| > **¿Problemas?** Consulta la [guía de solución de problemas](docs/content/docs/overview/troubleshooting.mdx) para incidencias comunes de instalación, generación, descarga de modelos y GPU. |
🧰 Tools
🪛 markdownlint-cli2 (0.22.1)
[warning] 105-105: Blank line inside blockquote
(MD028, no-blanks-blockquote)
[warning] 107-107: Blank line inside blockquote
(MD028, no-blanks-blockquote)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@README.es.md` around lines 104 - 108, Remove the blank lines inside the
blockquote in README.es.md so the callout stays as consecutive quoted lines and
satisfies markdownlint MD028. Keep the “Ver todos los binarios”, “Linux”, and
“¿Problemas?” lines directly adjacent with no empty quoted lines between them,
preserving the existing links and text.
Source: Linters/SAST tools
Tag the previously unlabeled fences flagged by markdownlint MD040: - 'claude mcp add voicebox ...' block -> bash - project structure tree block -> text Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.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.
What
Adds Spanish (
es) translations of the top-level documentation as separate*.es.mdfiles, with a reciprocal language switcher on each doc.Why
Lowers the entry barrier for Spanish-speaking users and contributors. Complements the UI locale work (#798) so both the app and its docs are available in Spanish. Kept as a separate PR from the UI locale so each is small and easy to review.
Changes
README.es.md,CONTRIBUTING.es.md,SECURITY.es.md— full translations. Code blocks, commands, paths, CLI flags, brand/model names, badges and table structure are preserved; only prose is translated. The README's internal anchors point to the translated headings.README.md/CONTRIBUTING.md/SECURITY.md— add anEnglish · Españolswitcher line near the top so each English doc links to its Spanish counterpart (and back).docs/AUTOUPDATER_QUICKSTART.md; repointed to the actualdocs/content/docs/developer/autoupdater.mdx(in both EN and ES).Verification
#anchorlinks and local file links resolve across all six files (checked programmatically).Notes
CHANGELOG.mdleft untouched — it's auto-compiled during the release workflow per the file header.docs/(Fumadocs) and the landing page are intentionally out of scope here — localizing those needs an i18n-architecture decision and can be follow-up PRs. Happy to open an issue to align on the approach if there's interest.Summary by CodeRabbit