Repository navigation
Conversation
…coded Windows path Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01THB2EzKrjM4G45oc46dP5k
|
Welcome to UniversalInvoiceMail! Thank you for opening your first pull request. Please ensure all tests pass and your changes adhere to local-first privacy invariants. |
|
Zweitmodell-Review (Cloud-Bot): Der hartcodierte Windows-Pfad wird durch einen repo-relativen Pfad ersetzt. Das ist ein korrekter CI-Fix mit einer Zeile und ohne Nebenwirkungen, freigabereif. Formales Approve war nicht möglich, weil Bot und Autor derselbe Account sind. Generated by Claude Code |
lukisch
left a comment
lukisch
left a comment
There was a problem hiding this comment.
Zweitprüfung (Cloud-Bot), Diff gelesen, Tests nicht lokal ausgeführt: Die Änderung ersetzt den hartcodierten Windows-Pfad C:/_Local_DEV/... durch Path(__file__).resolve().parent.parent / "locales" / "translations.json". Das ist korrekt (tests/ liegt direkt unter dem Repo-Root) und behebt den CI-Fehler auf #3. Keine Blocker, CI (2 Checks) grün. Freigabe/Merge bleibt beim Maintainer. Nebenbei: trans_file.open(...) wird nicht geschlossen, ein with-Block oder read_text wäre sauberer (nicht blockierend).
Generated by Claude Code
lukisch
left a comment
lukisch
left a comment
There was a problem hiding this comment.
Zweitprüfung: Der Test test_german_ui_texts_use_real_umlauts nutzte einen hartkodierten Windows-Pfad (C:/_Local_DEV/...) und schlug dadurch auf CI (test 3.10/3.11/3.12 auf #3) fehl. Die Auflösung über Path(__file__).resolve().parent.parent / "locales" / "translations.json" ist korrekt und plattformneutral. Nach dem Merge in den Basisbranch sollten die roten Checks auf #3 grün werden.
Generated by Claude Code
lukisch
left a comment
lukisch
left a comment
There was a problem hiding this comment.
Zweitmodell-Review (Freigabe empfohlen; Self-Approve nicht möglich): Der hartcodierte Windows-Pfad zu translations.json ist jetzt repo-relativ. Das behebt die roten test (3.10–3.12)-Checks auf #3.
Generated by Claude Code
lukisch
left a comment
lukisch
left a comment
There was a problem hiding this comment.
Zweitmodell-Review (LGTM): Hartcodierter C:/_Local_DEV/...-Pfad durch repo-relativen Pfad ersetzt; CI grün.
Generated by Claude Code
lukisch
left a comment
lukisch
left a comment
There was a problem hiding this comment.
Zweitmodell-Review: freigabereif (Formal-Approve nicht möglich, eigener PR). Hartkodierter Windows-Pfad durch repo-relativen Pfad ersetzt – korrekter CI-Fix für #3.
Generated by Claude Code
|
Review (Zweitmodell, Cloud-Bot): Repo-relativer Pfad statt hartcodiertem Generated by Claude Code |
test_german_ui_texts_use_real_umlautsreadC:/_Local_DEV/repos/UniversalInvoiceMail/locales/translations.json(hardcoded Windows path), causing FileNotFoundError on all CI runners (3.10/3.11/3.12). Now resolved relative to the test file. Locally verified the path exists and the JSON loads (70 keys). Full pytest not run here (no pytest/PySide6 in the sandbox); CI will confirm.Stacked on #3 — merge into its branch.
Generated by Claude Code