fix(tooling): translate-docs-check dwingt weer af (consistentie i.p.v. 31 talen) #1341
No reviewers
Labels
No labels
accepted
bug
declined
docs
duplicate
enhancement
good first issue
in-progress
needs-info
privacy
security
triage
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
LibreKAT/Ocideck!1341
Loading…
Reference in a new issue
No description provided.
Delete branch "fix/translate-docs-gate-enforce"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Vervolg op #1338. Toen die de crash weghaalde, bleken twee gebreken die de crash had gemaskeerd. Jij koos: docs blijven Engels + Nederlands, alleen de poort repareren (geen 270 machinevertalingen — dat past bij OciDeck's "inhoud = nl+en, interface = alle talen", en de lezer valt al netjes terug op de Engelse basis met een melding).
1. Tandeloze poort.
mainwasFuture<int>metreturn _runCheck(), maar de Dart-VM negeert een teruggegeven int — zonderexit()/exitCodeeindigde de poort altijd 0. Een dunnemainzet nuexitCode = await _run(args).2. Verkeerde eis → consistentie i.p.v. volledigheid. De check verlangde een variant in álle 31 talen. Nu bewaakt hij dat wat we verschepen kloppend is:
shippedDocLanguages = ['nl']) bestaat én is inpubspec.yamlgeregistreerd;PRIVACY.md,SECURITY_DESIGN.md) is ooit machinevertaald;shippedDocLanguageswordt tegenkLanguageNamesgevalideerd.Een taal erbij verschepen = code toevoegen aan
shippedDocLanguagesenmake translate-docsdraaien.Poort hing nergens in. Een gerepareerde poort die geen enkel aggregaat draait is nog steeds tandeloos, dus
translate-docs-checkstaat nu inSTATIC_GATES— draait per PR (static-gate.yml) i.p.v. alleen met de hand.Bewijs.
docVariantProblems()is publiek;test/doc_translation_test.dartdraait de poortlogica tegen de repo. Rood-eerst getoetst: een lossedocs/GLOSSARY.fr.mdlaat--checkmet exit 1 falen en de melding "untracked variant … add fr to shippedDocLanguages or remove the file"; schoon weer exit 0. Volledigemake checkgroen inclusief de nieuwe gate (8936 tests, coverage 87,1%).