[Matiiss]

They are outdated and mostly unmaintained
And they don't add much value to the project

Closes #3646

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 5206ed3a-4fc8-4c50-833b-560073122bc0

📥 Commits

Reviewing files that changed from the base of the PR and between 5de22d3 and c01c1cd.

📒 Files selected for processing (12)

• README.rst
• docs/readmes/README.de.rst
• docs/readmes/README.es.rst
• docs/readmes/README.fa.rst
• docs/readmes/README.fr.rst
• docs/readmes/README.gr.rst
• docs/readmes/README.it.rst
• docs/readmes/README.ja.rst
• docs/readmes/README.pt-br.rst
• docs/readmes/README.ru.rst
• docs/readmes/README.zh-cn.rst
• docs/readmes/README.zh-tw.rst

💤 Files with no reviewable changes (12)

• docs/readmes/README.gr.rst
• docs/readmes/README.fr.rst
• docs/readmes/README.zh-tw.rst
• docs/readmes/README.ja.rst
• docs/readmes/README.zh-cn.rst
• docs/readmes/README.it.rst
• docs/readmes/README.fa.rst
• docs/readmes/README.ru.rst
• docs/readmes/README.es.rst
• README.rst
• docs/readmes/README.pt-br.rst
• docs/readmes/README.de.rst

---

📝 Walkthrough

Walkthrough

Removes the multilingual language-navigation header from README.rst and deletes 11 localized README files under docs/readmes/ (German, Spanish, Persian, French, Greek, Italian, Japanese, Portuguese-BR, Russian, zh-cn, zh-tw), eliminating those translation documents and their anchor targets.

Changes

Cohort / File(s)	Summary
Main README header README.rst	Removed the multilingual localization header block and the 11 locale anchor definitions (language navigation anchors removed).
Deleted localized READMEs docs/readmes/README.de.rst, docs/readmes/README.es.rst, docs/readmes/README.fa.rst, docs/readmes/README.fr.rst, docs/readmes/README.gr.rst, docs/readmes/README.it.rst, docs/readmes/README.ja.rst, docs/readmes/README.pt-br.rst, docs/readmes/README.ru.rst, docs/readmes/README.zh-cn.rst, docs/readmes/README.zh-tw.rst	Removed entire localized README files (static documentation only). Reviewers should check for stale links or documentation references elsewhere that pointed to these files or anchors.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Translated README into German #3665 — Touches the same German README and README language anchors.
• Translate readme to Brazillian Portuguese #3582 — Touches the same Portuguese (Brazil) README and language anchors.
• Remove legacy credits from the Italian README #3713 — Related edits to the Italian README file.

Suggested reviewers

• oddbookworm
• ankith26

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Out of Scope Changes check	⚠️ Warning	The PR stays focused on removing README translation files; however, README.it.rst includes unrelated declarations about Calculator methods and variables that appear extraneous.	Remove the unrelated code declarations (coderabbit_add, coderabbit_formula signature change, and variable rename) that should not be part of this documentation-only PR.
Linked Issues check	❓ Inconclusive	The PR deletes all translated README files rather than implementing any of the three alternatives proposed in issue #3646, which partially diverges from the issue's suggested solutions.	Clarify whether deleting all translations aligns with the issue's intent or if it represents a decision to pursue a fourth alternative not originally discussed.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately and concisely describes the primary change: deleting README translations across multiple languages.
Description check	✅ Passed	The description explains the rationale for the changes and references the related issue, providing context for the deletions.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

📝 Coding Plan

• Generate coding plan for human review comments

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Tip

You can disable sequence diagrams in the walkthrough.

Disable the reviews.sequence_diagrams setting to disable sequence diagrams in the walkthrough.

[robotsnh]

@Matiiss I personally think that this allows those who don't speak English to add value to this project.

[oddbookworm]

@robotsnh sure, but what about the ongoing maintenance requirements of the translations? They're all in different levels of "up-to-date" with the English README because someone hasn't happened to swing by and drop an update

[oddbookworm]

Plus, all of the documentation is in English, so if someone doesn't have the English skills to read the readme, then I've got some unfortunate news in that the docs are going to be nearly impossible to read

[robotsnh]

@oddbookworm Fair enough. In addition, people can use Google Translate.

[oddbookworm]

I tend to agree with this move for the reasons mentioned already in the comments. But I think the decision to move forward with this should be left to the new steering council after the election

[aatle]

I agree with this change, pygame-ce's language translation does not currently extend past the README, and maintenance is very difficult.