Repository navigation
Conversation
…ntracked Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01THB2F1sVYAjmGPbgVKkNHq
|
Welcome! Thanks for your first pull request in this repository. A maintainer will review it soon. Please make sure:
Thanks for contributing! |
|
Zweitmodell-Review: Fix ist plausibel (MARKETING-LOG.txt ist bewusst untracked, Tests skippen nur bei Abwesenheit und laufen lokal weiter). Hinweis: Skip statt Assertion schwächt den Vertrag in CI – akzeptabel, da Datei per Policy lokal-only ist. Auf diesem Branch laufen keine CI-Checks (Base ist der #3-Branch); Merge nach #3 und dort die CI abwarten. Generated by Claude Code |
lukisch
left a comment
lukisch
left a comment
There was a problem hiding this comment.
Zweitmodell-Review (Cloud-Bot): Diff gelesen, keine Blocker.
- Vier MARKETING-LOG-Contract-Tests werden per
pytest.skipübersprungen, wenn die Datei fehlt. Das passt zu #3, das die Datei untracked. Lokal mit vorhandener Datei laufen alle Assertions unverändert. - Prüfen: Gibt es in
test_metadata.pyweitere Tests, dieMARKETING-LOG.txtlesen (z. B. auf Existenz oder Inhalt), ohne zu skippen? Sonst bleibt CI dort rot. Im Diff sind es genau die vier geänderten. - Nit:
import pyteststeht vor den Stdlib-Imports (re,pathlib). - Auf diesem Head lief nur
welcome. Die Verifikation kommt erst durch die CI nach dem Merge in #3. Ich habe nichts gemerged; die Freigabe bleibt beim Maintainer.
Generated by Claude Code
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01THB2F1sVYAjmGPbgVKkNHq
lukisch
left a comment
lukisch
left a comment
There was a problem hiding this comment.
Zweitmodell-Review (Freigabe empfohlen; Self-Approve nicht möglich): Die Tests, die MARKETING-LOG.txt verlangen, werden bei fehlender lokaler Datei übersprungen. Das behebt die 9 roten Smoke-&-Contract-Checks auf #3.
Generated by Claude Code
Fixes the red CI on #3: that PR untracks
MARKETING-LOG.txt(GITHUB-POLICY §4), but four contract tests intests/test_metadata.pystill asserted the file exists. They nowpytest.skipwhen it is absent and still run locally where the file exists.Verified:
tests/test_metadata.py→ 26 passed, 4 skipped. Target is the #3 branch; merge this into it to turn #3 green.Generated by Claude Code