Repository navigation
Conversation
…, harden translator app_dir & contract tests [G 2026-11-02]
|
Welcome! Thanks for your first pull request in this repository. A maintainer will review it soon. Please make sure:
Thanks for contributing! |
|
Zweitmodell-Review (CI-Diagnose): Die 9 roten Smoke-&-Contract-Jobs kommen ausschließlich von Empfehlung: #4 zuerst mergen (oder in #3 einbinden) und #3 darauf rebasen. Danach sollte CI grün sein. Am Code selbst habe ich in diesem Lauf nichts beanstandet. Generated by Claude Code |
|
CI-Diagnose (Cloud-Bot): Alle 9 Smoke-&-Contract-Jobs scheitern mit Exit 1 (Logs sind nicht mehr abrufbar). Wahrscheinliche Ursache: Contract-Tests erwarten die lokale Generated by Claude Code |
lukisch
left a comment
There was a problem hiding this comment.
Zweitreview (Cloud-Bot), Head 14b76df: so nicht mergebar, Neuaufbau auf master empfohlen.
Die vier roten Tests (MARKETING-LOG.txt must exist) sind bekannt und über #4 adressiert. Neu aus diesem Durchgang:
- Merge-Konflikte mit
master(ee7b311, vier Commits weiter) in sechs Dateien:CHANGELOG.md,README.md,README_de.md,llms.txt,tests/test_metadata.pyundMARKETING-LOG.txt(aufmasterin1f0e7cageändert, hier gelöscht). #4 löst das nicht. - Beschädigter Text in
CHANGELOG.md:8und:11. Die Shell hat Backslash-Folgen aus Backticks gefressen: ein echtes TAB gefolgt vonranslator.py, ein BEL-Zeichen (0x07) vorpp_dir, TAB +ests/test_metadata.py, TAB +est_internal_marketing_log_not_tracked. Die PR-Beschreibung hat denselben Schaden. - Testzahlen widersprechen sich. CHANGELOG und PR-Text sagen „157 passed", die README-Badges hier 155,
master162. Gemessen (Linux, Python 3.12): dieser Head 4 failed, 149 passed, 4 skipped; #4-Head 149 passed, 8 skipped;master158 passed, 4 skipped. - Toter Verweis —
.github/labels.yml:2nenntlabel-sync.yml, den Workflow gibt es nicht. Die Datei bewirkt nichts. - Skip statt Vertrag (Entwurfsfrage). Mit #4 werden die vier Marketing-Log-Tests in jedem CI-Lauf übersprungen und prüfen nur noch auf dem Rechner des Autors etwas.
- Kosmetik: Der Docstring in
translator.py:35sagt noch „default: aktuelles Verzeichnis", obwohl:40das geändert hat. Die Änderung selbst ist in Ordnung.
Empfehlung: Der eigentliche Inhalt ist klein (Datei aus dem Tracking nehmen, .gitignore, eine Zeile in translator.py, zwei YAML-Dateien, zwei Tests). Frisch von master neu aufbauen, die Skips aus #4 gleich einarbeiten, CHANGELOG reparieren, Badge auf die dann gemessene Zahl setzen und #3 sowie #4 als ersetzt schließen. Das ist einfacher als ein Rebase über sechs Konflikte.
Generated by Claude Code
Summary of Changes