Conversation
|
Reviewed on-device, looks good to my german eyes. |
|
Thanks @bhelm. Reviewed the whole delta. The headline: the red check is not yours, and the two code changes are both genuine bug fixes. One blocking item on the new test file. The failing check
I merged current main into your head locally: clean, zero conflicts. On that merged tree: A rebase onto main clears the red. Nothing to change for it. The two code changes are real fixesAndroid, uiString(R.string.l10n_today_screen_baselines_learned_on_device_over_14_359f6812) +
" signal against a typical adult range, not medical advice."So every locale rendered that sentence half-translated, not just German. Routing the whole sentence through iOS, Blocking: the new test file never runs
Worth doing rather than dropping, because one of those 8 would have caught the next item. Non-blockingGerman terminology splits on the exact concept this PR is about. After the change:
Two adjacent VoiceOver labels for the same thing now disagree, and Android disagrees with iOS. Your 8 dead strings. What I checked mechanically
What I cannot checkGerman prose quality. The terminology table above is a consistency observation, not a judgement on which wording reads better, and I will take your call on it. Rebase, convert the test, and I will merge. |
|
Thanks for the review. The German Today test is now a German sync counts now say Datenblock / Datenblöcke everywhere: both iOS sync keys, Android The unused |
The file was pytest-style under Tools/tests/, which no workflow runs with pytest and the core Tools job does not collect, so its eight checks never executed. Convert it to unittest.TestCase (TemporaryDirectory for the former tmp_path case) and move it next to the other i18n suites in Tools/, where the core Tools job collects it. Raise the core floor from 108 to 117, the exact count now collected, so deleting the suite turns the job red.
The German sync chip said Datenblock/Datenblöcke on the new iOS key but Chunk/Chunks on the iOS pages variant and the Android plural. Use Datenblock/Datenblöcke in all three and pin them in the German Today test.
The Android sync note shown on Today, Health, Sleep and Intelligence read "Brocken gezogen", a literal translation, while the iOS twin already says "Datenblöcke übertragen". Use the iOS wording and pin it.
Today now renders today_recovery_vitals_explanation as one resource, so l10n_today_screen_baselines_learned_on_device_over_14_359f6812 has no reader left in Kotlin, Swift, tests or Tools. Remove it from the base and all seven translated strings.xml files.
d9d82ac to
02b346c
Compare
Summary
German language concept
Verification
python3 Tools/tests/test_german_today_localization.pypython3 Tools/test_home_i18n.pypython3 Tools/test_i18n_audit.pypython3 Tools/i18n_audit.py --ci origin/main./android/gradlew -p android :app:testFullDebugUnitTest --tests com.noop.ui.GermanLocalizationTest