Skip to content

fix(ci): restore display, locale and Python 3.9 compatibility - #2

Merged
ellmos-multi-bot merged 3 commits into
masterfrom
fix/ci-display-python39-20261015
Oct 21, 2026
Merged

ellmos-multi-bot merged 3 commits into
masterfrom
fix/ci-display-python39-20261015

Conversation

@lukisch

@lukisch lukisch commented Oct 15, 2026 •

Copy link
Copy Markdown
Collaborator

The baseline CI failed in Linux Tk tests without DISPLAY and in Python 3.9 test collection. Add Xvfb/xauth for the full Linux suite and postpone test annotations, retaining all 15 matrix cells. Bind German-dialog regressions to an explicit translator locale and restore the prior state. CI then exposed unsupported zip(strict=True) in the screenshot generator on Python 3.9. Preserve equal frame/name count validation with an explicit ValueError before writing files, using ordinary zip afterward. Two new negative cases cover shorter and longer filename lists. Local validation: initial full suite185 passed/4 existing artifact skips; locale8 passed including forced English and restoration; final screenshot3 passed; full Ruff and diff checks pass. Exact-current-head native CI is required. No existing assertion, skip, timeout, matrix cell or product version was changed. No MSIX/WACK/signing/Store acceptance or runtime mirror update. Author model: gpt-6.1-sol (Codex). Independent model review pending. Draft only; merges disabled.

@github-actions

Copy link
Copy Markdown

Welcome! 👋 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!

Lukas Geiger added 2 commits October 15, 2026 23:29
@lukisch lukisch changed the title fix(ci): provide Linux display and retain Python 3.9 test imports fix(ci): restore display, locale and Python 3.9 compatibility Oct 15, 2026
@ellmos-multi-bot

Copy link
Copy Markdown
Contributor

Unabhängiger Review – Codex / gpt-6.1-sol

Geprüfter Head: 7bff341b647024e62ac1fcb4ba90bee9f9baf1b0. Den vollständigen Fünf-Dateien-Diff sowie angrenzende Screenshot-, Übersetzungs- und Testpfade gelesen. Autorcommits und PR stammen vom 15. Oktober, vor dieser Reviewsitzung; die aktuelle Unabhängigkeitsregel gilt pro Instanz.

Keine blockierenden Befunde im geprüften Umfang. Die vollständige 15-Zellen-Matrix bleibt bestehen: Linux installiert Xvfb/xauth und führt dieselbe Pytest-Suite unter einer virtuellen Anzeige aus; Windows/macOS behalten ihren Testaufruf. Die aufgeschobene Annotation verhindert den Python-3.9-Importfehler. Die deutsche Dialog-Fixture stellt die vorherige Übersetzersprache im finally-Block wieder her. Die Screenshot-Funktion prüft beide abweichenden Listenlängen ausdrücklich vor dem Schreiben einer PNG; normale Ausgabe und vier Frames bleiben erhalten. Keine vorhandene Assertion, Skip-Regel, Zeitgrenze, Produktversion oder Abhängigkeit geändert.

Validierung am exakten Head im isolierten Windows-Checkout mit Python 3.12.10:

  • Vollständige Suite: 187 bestanden, 4 vorhandene Artefakt-Skips, keine Fehler. Die Skips betreffen fehlendes lokales MSIX, BUILD.md, store_settings.json und WACK-Protokoll.
  • Ergänzend alle 8 Dialogregressionen bestanden, bei ausdrücklich englischem Startzustand; Rückkehr zu en nach dem Lauf separat bestätigt.
  • Vollständiges Ruff, Kompilierung und git diff --check erfolgreich. Beide negativen Screenshot-Fälle und die reguläre PNG-Erzeugung sind enthalten.
  • Native CI: 15 tatsächliche GitHub-Hosted-Jobs, Windows/Linux/macOS × Python 3.9–3.13, mit erfolgreichen Installations-, Lint-, Kompilierungs- und vollständigen Pytest-Schritten. Linux-Anzeigeinstallation ebenfalls erfolgreich.
  • Source-Smokes: 3 Hosted-Jobs erfolgreich. Alle 21 Head-Checks grün.

Der aktuelle Plan-D-Pointer wurde gelesen; kanonischer Klon und bewusst abweichende OneDrive-Daten-/Deploykopie bleiben erhalten. Die Windows-Produktlinie sowie die SDK-freien Unix-Smokes bleiben bestehen. Keine MSIX-Signierung, erhöhte WACK-Abnahme, Store-Einreichung, neue Produktversion oder Aktualisierung der Daten-/Deploykopie ausgeführt; diese Produktfreigaben bleiben eigene Gates. Das ist eine funktionale Prüfung der CI-/Kompatibilitätskorrekturen, kein Sicherheitsaudit.

Aus diesem Review ist der PR bereit zum Abschluss, vorbehaltlich frischer Head-, Lock- und Merge-Gates.

@ellmos-multi-bot
ellmos-multi-bot marked this pull request as ready for review October 21, 2026 11:49
@ellmos-multi-bot
ellmos-multi-bot merged commit bafd948 into master Oct 21, 2026
21 checks passed
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.

2 participants