Skip to content

fix(deps): exclude PySide6 6.12.0 (CI abort on Linux py3.10/3.11) - #5

Open
lukisch wants to merge 3 commits into
chore/hygiene-ci-dependabot-sbom-20261031from
claude/serene-cannon-oxs8xm
Open

lukisch wants to merge 3 commits into
chore/hygiene-ci-dependabot-sbom-20261031from
claude/serene-cannon-oxs8xm

Conversation

@lukisch

@lukisch lukisch commented Oct 31, 2026

Copy link
Copy Markdown
Collaborator

Problem

CI of #4: Pytest - Python 3.10/3.11 on ubuntu-latest fail. 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) in requirements.txt and pyproject.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

@github-actions

Copy link
Copy Markdown

Welcome to UniversalDocsGrabber! 👋 Thanks for your first pull request in this repository.

A maintainer will review it soon. Please make sure:

  • Your changes are tested
  • Documentation is updated if needed
  • The PR description explains what and why

Thanks for contributing!

@lukisch lukisch left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zweitmodell-Review: keine Blocker.

  • PySide6>=6.5.0,!=6.12.0 steht konsistent in pyproject.toml und requirements.txt. Das entspricht dem Fix, der in CleanMarkdown auf main bereits 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.toml entsteht.

Freigabe/Merge bleibt beim Maintainer.


Generated by Claude Code

@lukisch lukisch left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 lukisch left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 lukisch left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 lukisch left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 lukisch left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 lukisch left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 lukisch left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 lukisch left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant