Skip to content

Comments

chore: Sync translations#2918

Open
github-actions[bot] wants to merge 1 commit intodevfrom
feat/translations
Open

chore: Sync translations#2918
github-actions[bot] wants to merge 1 commit intodevfrom
feat/translations

Conversation

@github-actions
Copy link
Contributor

Sync translations from crowdin.com/project/revanced

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR syncs translations from Crowdin for the ReVanced Manager Android application. It adds or updates string resource files for numerous languages.

Changes:

  • Added translation files for 50+ language variants
  • Added substantial translations for Chinese (Traditional and Simplified), Vietnamese, Ukrainian, Turkish, Thai, Swedish, and Serbian
  • Added placeholder files with empty resources for many other languages

Reviewed changes

Copilot reviewed 53 out of 77 changed files in this pull request and generated no comments.

Show a summary per file
File Description
values-zh-rTW/strings.xml Added 470 Traditional Chinese translation strings
values-zh-rCN/strings.xml Added 460 Simplified Chinese translation strings
values-vi-rVN/strings.xml Added 464 Vietnamese translation strings
values-uk-rUA/strings.xml Added 455 Ukrainian translation strings
values-tr-rTR/strings.xml Added 471 Turkish translation strings
values-th-rTH/strings.xml Added 455 Thai translation strings
values-sv-rSE/strings.xml Added 461 Swedish translation strings
values-sr-rSP/strings.xml Added 450 Serbian translation strings
values-my-rMM/strings.xml Added 2 Burmese translation strings
values-hr-rHR/strings.xml Added 17 Croatian translation strings
values-hi-rIN/strings.xml Added 3 Hindi translation strings
values-zu-rZA, values-uz-rUZ, etc. Added 30+ empty placeholder translation files for future translations

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Member

@validcube validcube left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥞 Translation looks fine to me, personally I haven't seen any abuse and Copilot didn't have anything to flags.

But this first time PR for ReVanced Manager Compose should be checked with Crowdin maintainer @Ushie to double verify configuration.

@Ushie
Copy link
Member

Ushie commented Jan 12, 2026

I think we should set a criteria to the amount of strings required to be translated to be synced

@Axelen123
Copy link
Member

Agreed. Otherwise, the app will show options for translations that don't exist or are very incomplete.

@oSumAtrIX
Copy link
Member

Imo this is too advanced. Once translations are finished more or less, this functionality wouldnt really have any use.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants