Skip to content

fix(ci): install image test dependencies and make contracts portable - #9

Draft
lukisch wants to merge 2 commits into
masterfrom
fix/ci-pillow-tests-20261015
Draft

lukisch wants to merge 2 commits into
masterfrom
fix/ci-pillow-tests-20261015

Conversation

@lukisch

@lukisch lukisch commented Oct 15, 2026 •

Copy link
Copy Markdown
Collaborator

The baseline CI stopped collecting image tests because Pillow was missing. Install it in the existing regression step. Once collection succeeded, CI exposed Python3.10 tomllib import, an asset filename with wrong Git case, and two Windows shortcut tests depending on a real Windows SystemRoot/notepad.exe. Add conditional test-only tomli, use the actual tracked SoftwareCenter.ico spelling, and exercise Windows percent-variable expansion against real temporary files using ntpath on every host. The file checks remain real; the first case now also rejects the deleted target. Runtime code, version, skips, matrix and existing policy remain unchanged. Baseline CI36245702422; first repair CI38812791709 identified these latent failures. Local final full suite on Windows Python3.12/offscreen Qt: 270 passed,5 existing skips; full Ruff and diff checks pass. Exact-current-head CI remains required. No Store, deployment or runtime mirror acceptance. Author model: gpt-6.1-sol (Codex). Independent model review pending. Draft only; merges disabled.

Generated-with: gpt-6.1-sol (Codex)
@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!

Generated-with: gpt-6.1-sol (Codex)
@lukisch lukisch changed the title fix(ci): install Pillow for existing asset tests fix(ci): install image test dependencies and make contracts portable Oct 15, 2026
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