Skip to content

Update translations and fix test#1871

Merged
Flow86 merged 3 commits intoReturn-To-The-Roots:masterfrom
Flamefire:translations
Jan 23, 2026
Merged

Update translations and fix test#1871
Flow86 merged 3 commits intoReturn-To-The-Roots:masterfrom
Flamefire:translations

Conversation

@Flamefire
Copy link
Member

Includes Return-To-The-Roots/languages#41
fixing a crash where the German translation was missing a format string.

Adapt the test to catch this:
If we don't "translate" something in the en_GB file it won't show upin the .mo file and won't be checked.

As the test checks all translated strings (see the continue) anyways iterate over the translation file instead, backreferencing the "gold" entries.

Includes Return-To-The-Roots/languages#41
fixing a crash where the German translation was missing a format string.

Adapt the test to catch this:
If we don't "translate" something in the `en_GB` file it won't show up
in the `.mo` file and won't be checked.

As the test checks all translated strings (see the `continue`) anyways
iterate over the translation file instead, backreferencing the "gold"
entries.
@Flamefire Flamefire requested a review from Flow86 January 23, 2026 10:44
The comparison is ambiguous, fixed in GCC 10.
@Flow86 Flow86 merged commit af47028 into Return-To-The-Roots:master Jan 23, 2026
18 checks passed
@Flamefire Flamefire deleted the translations branch January 23, 2026 21:16
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.

2 participants