Conversation
Add Hebrew translation dictionaries for app, ui, and desktop packages, register `he` in the desktop native locales and RTL set, and cover it in the i18n parity tests. 🤖 Generated with Codebuff Co-Authored-By: Codebuff <noreply@codebuff.com>
|
The following comment was made by an LLM, it may be inaccurate: Potential Duplicate FoundPR #39423: feat(i18n): Add Hebrew language support with RTL handling This appears to be a duplicate of the current PR #45205. Both PRs are adding Hebrew ( |
Add translation guidance for Hebrew: preferred terms, RTL handling, and the plural-form rules (one/two/other) used by the parity tests. 🤖 Generated with Codebuff Co-Authored-By: Codebuff <noreply@codebuff.com>
|
Automated PR Cleanup Thank you for contributing to opencode. Due to the high volume of PRs from users and AI agents, we periodically close older PRs using automated criteria so maintainers can focus review time on the most active and community-supported contributions. This PR was closed because it matched the following cleanup criteria:
PRs created within the last month are not affected by this cleanup. If you believe this PR was closed incorrectly, or if you are still actively working on it, please leave a comment explaining why it should be reopened. A maintainer can review and reopen it if appropriate. Thanks again for taking the time to contribute. |
Issue for this PR
Closes #42447 (feature request for Hebrew locale)
Type of change
What does this PR do?
Adds Hebrew (
he) support across the app, ui, and desktop packages:heinDESKTOP_NATIVE_LOCALES/ labels / tags (desktop-native.ts) and in the desktop renderer i18n (renderer/i18n/index.ts)heto theRTL_LOCALESset inlanguage.tsxso Hebrew renders right-to-leftheto the i18n parity tests (parity.test.ts,desktop-native.test.ts)packages/app/src/i18n/he.ts,packages/ui/src/i18n/he.ts,packages/desktop/src/renderer/i18n/he.ts.opencode/glossary/he.mdHebrew plural variants (
.two) followIntl.PluralRules("he-IL")categories (one,two,other).Supersedes #39423 (stale Hebrew PR with merge conflicts since July; the author did not respond). All work here is rebased on the latest
devwith no conflicts.How did you verify your code works?
parity.test.ts,desktop-native.test.ts): every English key covered, no unexpected extras, placeholders preserved, required plural variants presentdevwith no conflictsScreenshots / recordings
N/A (language support, no UI screenshots)
Checklist