Repository navigation
Conversation
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01THB2EyrSiWW2YqW8XR6oFP
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01THB2EyrSiWW2YqW8XR6oFP
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01THB2EyrSiWW2YqW8XR6oFP
|
Welcome to UniversalDocsGrabber! 👋 Thanks for your first pull request in this repository. A maintainer will review it soon. Please make sure:
Thanks for contributing! |
lukisch
left a comment
There was a problem hiding this comment.
Zweitmodell-Review: keine Blocker.
PySide6>=6.5.0,!=6.12.0steht konsistent inpyproject.tomlundrequirements.txt. Das entspricht dem Fix, der in CleanMarkdown aufmainbereits steht (ed46284).- Das nachgezogene Newline am Dateiende ist harmlos.
- CI (Ubuntu und macOS smoke) ist grün.
- Hinweis: PR #4 (CI-Matrix/Hygiene) berührt vermutlich dieselben Dateien. Bitte die Merge-Reihenfolge klären, damit kein Konflikt in
pyproject.tomlentsteht.
Freigabe/Merge bleibt beim Maintainer.
Generated by Claude Code
lukisch
left a comment
There was a problem hiding this comment.
Zweitmodell-Review: Änderung korrekt und konsistent (PySide6>=6.5.0,!=6.12.0 in pyproject.toml und requirements.txt; zusätzlich nur Newline am Dateiende). Smoke-Checks Ubuntu und macOS grün. Aus Reviewsicht merge-bereit. Formale Freigabe nicht möglich, da eigener PR.
Generated by Claude Code
lukisch
left a comment
There was a problem hiding this comment.
Zweitmodell-Review: freigabereif (Formal-Approve nicht möglich, eigener PR). PySide6-Ausschluss 6.12.0 konsistent in pyproject.toml und requirements.txt, Checks grün. Hinweis: Basis ist chore/hygiene-ci-dependabot-sbom-20261031 (#4, rot wegen 2 Fehlern, vermutlich genau dieser PySide6-Abbruch) – diesen Fix zuerst mergen, dann #4 neu laufen lassen.
Generated by Claude Code
lukisch
left a comment
There was a problem hiding this comment.
Zweitmodell-Review: PySide6-Constraint !=6.12.0 plus Newline am Dateiende, CI grün. Freigabefähig (formales Approve nicht möglich: eigener PR).
Generated by Claude Code
lukisch
left a comment
There was a problem hiding this comment.
Zweitmodell-Review: PySide6-6.12.0-Ausschluss plus Dateiende-Newline, unkritisch. Freigabefähig (formale Approval nicht möglich, gleicher Account wie Autor).
Generated by Claude Code
lukisch
left a comment
There was a problem hiding this comment.
Zweitmodell-Review: LGTM. Der Ausschluss von PySide6 6.12.0 ist konsistent mit MailProcessor#12 und PrivacyMailDesk#12, und alle 4 Checks sind grün. Die Newline am Dateiende in pyproject.toml ist harmlos. Empfehlung: mergen.
Generated by Claude Code
lukisch
left a comment
There was a problem hiding this comment.
Zweitprüfung: ohne Befund. PySide6!=6.12.0 konsistent, Smoke-Checks grün. Hinweis: gestapelt auf PR #4, erst nach dessen Merge relevant für main.
Generated by Claude Code
lukisch
left a comment
There was a problem hiding this comment.
Cloud-Review (Zweitmodell): Pin !=6.12.0 korrekt in pyproject.toml und requirements.txt; EOF-Newline-Fix harmlos. Vorhandene Checks (Smoke ubuntu/macos) grün, stammen aber vom 31.10. – vor dem Merge bitte CI auf aktuellem Head neu laufen lassen, da der Fehler (Linux py3.10/3.11) damit erst belegt behoben ist. Sonst freigabefähig.
Generated by Claude Code
lukisch
left a comment
There was a problem hiding this comment.
Zweitmodell-Review (LGTM): Der Pin PySide6>=6.5.0,!=6.12.0 ist in beiden Dateien konsistent, außerdem wurde das fehlende Newline am Ende von pyproject.toml ergänzt. Keine Einwände.
Generated by Claude Code
Problem
CI of #4:
Pytest - Python 3.10/3.11 on ubuntu-latestfail. All 232 tests pass, then the interpreter aborts at shutdown (Fatal Python error: none_dealloc ... refcount error in a C extension, exit 134).Fix
Same as doc-bricks/CleanMarkdown#11: exclude PySide6 6.12.0 (
!=6.12.0) inrequirements.txtandpyproject.toml.Caveat
Not verified locally (no PySide6 6.12.0 env here). The root cause is inferred from the identical abort signature; the CI run on this PR is the check. The same signature appears in doc-bricks/MailProcessor#9, which may deserve the same pin.
This PR targets the #4 branch, not master. Merge or cherry-pick as you prefer.
Generated by Claude Code