docs(i18n): add Indonesian translations#579
Conversation
📝 WalkthroughWalkthroughA single link to the Bahasa Indonesia translation ( ChangesREADME Language Switcher Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~1 minute Possibly related issues
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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.md`:
- Line 34: The Indonesian translation link was added to the main README on line
34 pointing to a file that does not yet exist, and the existing translation
files have not been updated with this new language link. Either create the
missing i18n/README.id.md translation file and add the Indonesian language link
(with appropriate anchor text and href to i18n/README.id.md) to the language
switcher section in all eight existing translation files (README.nl.md,
README.es.md, README.hi.md, README.zh.md, README.ta.md, README.bn.md,
README.fr.md, and README.de.md) to keep them synchronized, or remove the
Indonesian link that was added to the main README until the translation file and
all language switcher updates are ready.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
| <a href="i18n/README.bn.md">বাংলা</a> | | ||
| <a href="i18n/README.zh.md">中文</a> | | ||
| <a href="i18n/README.nl.md">Nederlands</a> | | ||
| <a href="i18n/README.id.md">Bahasa Indonesia</a> | |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Verify that i18n/README.id.md exists and that all i18n files have been updated
# Check if Indonesian translation file exists
echo "=== Checking for i18n/README.id.md ==="
if [ -f "i18n/README.id.md" ]; then
echo "✓ i18n/README.id.md exists"
else
echo "✗ i18n/README.id.md NOT FOUND"
fi
# Check if all i18n README files have been updated with Indonesian link
echo ""
echo "=== Checking for 'Bahasa Indonesia' in all i18n README files ==="
fd 'README\..+\.md$' i18n/ -x sh -c '
file="{}"
if grep -q "Bahasa Indonesia" "$file"; then
echo "✓ $file has Bahasa Indonesia link"
else
echo "✗ $file MISSING Bahasa Indonesia link"
fi
'Repository: GetBindu/Bindu
Length of output: 593
Indonesian translation file missing and language links not synchronized across all versions.
The Indonesian link added to the main README (line 34) points to i18n/README.id.md, which does not exist. Additionally, all eight existing translation files (i18n/README.nl.md, i18n/README.es.md, i18n/README.hi.md, i18n/README.zh.md, i18n/README.ta.md, i18n/README.bn.md, i18n/README.fr.md, i18n/README.de.md) are missing the Indonesian language link in their language switchers.
Before merging, either create the Indonesian translation file and update all language switchers across all eight translation files to include it, or remove the Indonesian link from the main README until both are ready.
🤖 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.md` at line 34, The Indonesian translation link was added to the main
README on line 34 pointing to a file that does not yet exist, and the existing
translation files have not been updated with this new language link. Either
create the missing i18n/README.id.md translation file and add the Indonesian
language link (with appropriate anchor text and href to i18n/README.id.md) to
the language switcher section in all eight existing translation files
(README.nl.md, README.es.md, README.hi.md, README.zh.md, README.ta.md,
README.bn.md, README.fr.md, and README.de.md) to keep them synchronized, or
remove the Indonesian link that was added to the main README until the
translation file and all language switcher updates are ready.
1 participant
Summary
Describe the problem and fix in 2–5 bullets:
Change Type (select all that apply)
Scope (select all touched areas)
Linked Issue/PR
User-Visible / Behavior Changes
Added Indonesian to the list of languages in the README. No functional changes.
Security Impact (required)
No)No)No)No)No)Yes, explain risk + mitigation:Verification
Environment
Steps to Test
Expected Behavior
Actual Behavior
Evidence (attach at least one)
Human Verification (required)
What you personally verified (not just CI):
Compatibility / Migration
No)No)No)Failure Recovery (if this breaks)
Risks and Mitigations
List only real risks for this PR. If none, write
None.Checklist
uv run pytest)uv run pre-commit run --all-files)Summary by CodeRabbit