Conversation
|
Welcome! Thanks for your first pull request in this repository. A maintainer will review it soon. Please make sure:
Thanks for contributing. |
lukisch
commented
Oct 3, 2026
lukisch
left a comment
Collaborator
Author
There was a problem hiding this comment.
Zweitmodell-Review: CI grün (12/12), keine Blocker.
- Tooltips laufen jetzt über
t(...), die gespeicherte Sprache wird beim Start überconfigure_application_language()wiederhergestellt (ungültige oder nicht hashbare Werte fallen aufdezurück, per Test abgedeckt). manage_translations.pyerkenntt("...")-Aufrufe per AST. Das Skript parst nur und führt keinen Code aus.- Keine Credentials oder Nutzerpfade im Diff.
Hinweise: Der Test importiert from src.main import ..., während der Rest gui.*/core.* über src/ im Pfad nutzt. Das funktioniert hier, ist aber ein zweiter Importpfad für dasselbe Modul. Außerdem ändert dieser PR src/main.py und translations.json wie #13/#14/#15, bitte Merge-Reihenfolge beachten.
Generated by Claude Code
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Die Sprachauswahl im Einstellungsdialog zeigte bisher immer Deutsch und speicherte Änderungen nicht. Beim Start wurde die gespeicherte Sprache außerdem nicht an den Translator übergeben. Der Dialog lädt und speichert die Auswahl jetzt; nach einem Neustart verwenden die angebundenen Hilfetexte und Qt-Standarddialoge diese Sprache. Ungültige gespeicherte Werte fallen auf Deutsch zurück.
t(...)-Aufrufe unabhängig von deutschen Sprachmerkmalen und meldet Katalogvollständigkeit präzise.Validierung am Head
ccb2334f7f7242d4f6a3a552570f48ac59098122: Gesamtsuite 391 bestanden / 2 Skips, 93,35 s, Prozess-Exit 0; gezielt 23 Sprach-/i18n-Tests, Exit 0; Ruff, Syntax- und Diff-Prüfung grün. Die Skips betreffen eine fehlende lokale Release-EXE und den PNG-Smoke unter einer nativen Qt-Plattform. Übersetzungsprüfung Exit 0: 320 vollständige Katalogeinträge, 157 direkte Literal-t-Schlüssel abgedeckt, echte Umlaute und keine Unicode-Ersatzzeichen.Unabhängiges Review im separaten Worktree: Diff und alle 87 neuen Übersetzungen geprüft, 23 Sprach-/i18n-Tests bestanden (4,45 s, Prozess-Exit 0), keine offenen Delta-Findings. Ein Scanner-Befund zu fünf heuristisch übersehenen Schlüsseln wurde aufgenommen und durch einen Regressionstest abgesichert.
Der geschützte
masterbenötigt weiterhin eine GitHub-Freigabe. Bei späterer Integration mit PR #14/#15 die Sprachinitialisierung und die dortigen GC-/Worker-Abschlusskorrekturen gemeinsam erhalten.Finale CI am unveränderten Head: alle neun Matrixjobs erfolgreich, CodeQL erfolgreich.