Skip to content

ci: exercise declared macOS support with native tests - #11

Merged
ellmos-multi-bot merged 1 commit into
mainfrom
codex/mac-ci-20261016
Oct 21, 2026
Merged

ellmos-multi-bot merged 1 commit into
mainfrom
codex/mac-ci-20261016

Conversation

@lukisch

@lukisch lukisch commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

The package declares macOS support, while CI only exercises Linux and Windows. Add macos-latest to the existing Python 3.10-3.13 matrix, running the same installation, lint and full source suite on the missing native platform. Existing permissions, timeout and Linux/Python 3.13 distribution build stay unchanged.

Validation: all 38 existing tests pass locally on native Windows/Python 3.12, including CLI, SQLite and temporary loopback HTTP tests. Ruff, compileall and diff check pass. Exact-head native CI remains required; these tests do not certify interactive browser use on a physical Mac.

One workflow line changes; product code, tests, dependencies, version and runtime mirrors are unchanged. Author model: gpt-6.1-sol (Codex). Draft only; independent different-model review pending, merges disabled.

Add macOS to the existing Python 3.10-3.13 source-test matrix while preserving all current gates, permissions and Linux distribution build.

Generated-with: gpt-6.1-sol (Codex)
@ellmos-multi-bot

Copy link
Copy Markdown
Contributor

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

Geprüfter Head: e751721d101401d5bab17354243d24af15206786. Der vollständige Diff (eine Zeile in .github/workflows/ci.yml), der gesamte Workflow und angrenzende Projekt-/Paketvorgaben wurden gelesen. Der Autorcommit vom 15.10. liegt vor dieser Reviewsitzung; die aktive Unabhängigkeitsregel gilt pro Instanz.

Die Änderung ergänzt macos-latest zur vorhandenen Linux-/Windows-Matrix für Python 3.10–3.13. Dieselben Installations-, Ruff- und vollständigen Testschritte laufen auf allen zwölf Kombinationen. Zeitlimit, fail-fast-Einstellung, read-only Berechtigungen, gepinnte Actions und der Linux-/Python-3.13-Distributionsbau bleiben erhalten. Keine funktionalen Befunde im geprüften Umfang.

Validierung am exakten Head:

  • Lokal Windows/Python 3.12.10: 38 Tests bestanden, keine Fehler, Fehlschläge oder Skips. Vorhandene CLI-, SQLite-, Sprach- und temporäre lokale HTTP-Regressionen vollständig ausgeführt. Ruff, Kompilierung, Wheel-/sdist-Bau und git diff --check erfolgreich. Isolierter eigener Checkout; echte Nutzerdaten und installierte Anwendung unverändert.
  • Native CI: 12/12 tatsächliche GitHub-Hosted-Jobs erfolgreich, Linux/Windows/macOS × Python 3.10–3.13. Runner-Labels und erfolgreiche Install-/Lint-/Test-Schritte zurückgelesen; Distributionsbau auf Ubuntu/Python 3.13 ebenfalls erfolgreich. Alle 28 Head-Checks grün (zwei vorhandene CI-Läufe am selben Head plus weitere Checks).

Der CI-only-PR ist aus diesem unabhängigen Review bereit zum Abschluss, vorbehaltlich frischer Head-, Lock- und Merge-Gates im aktiven Betreiberprofil. Interaktive Browsernutzung auf einem physischen Mac wird damit nicht als abgenommen behauptet; sie ist nicht Gegenstand dieser Matrix-Ergänzung. Keine Sicherheitsanalyse oder Produktveröffentlichung durchgeführt.

@ellmos-multi-bot
ellmos-multi-bot marked this pull request as ready for review October 21, 2026 09:58
@ellmos-multi-bot
ellmos-multi-bot merged commit a407f7b into main Oct 21, 2026
28 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