Skip to content

Run pytest across the supported Python CI matrix - #522

Merged
iMicknl merged 3 commits into
mainfrom
codex/add-pytest-ci
Aug 16, 2026
Merged

Run pytest across the supported Python CI matrix#522
iMicknl merged 3 commits into
mainfrom
codex/add-pytest-ci

Conversation

@iMicknl

@iMicknl iMicknl commented Aug 10, 2026

Copy link
Copy Markdown
Owner

Summary

  • run the pytest suite in every supported Python-version CI job
  • key the cached virtual environment by operating system and Python version
  • keep linting and type checks in the same existing workflow

Verification

  • Full local test suite
  • Repository hooks

This is a draft while the new CI matrix runs on GitHub.

@iMicknl
iMicknl marked this pull request as ready for review August 16, 2026 17:47
Copilot AI lite review requested due to automatic review settings August 16, 2026 17:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the primary GitHub Actions workflow to execute the project’s pytest suite across the supported Python CI matrix, while improving cache isolation to avoid cross-environment reuse.

Changes:

  • Run pytest in the existing matrix job for each supported Python version.
  • Key the .venv cache by OS + Python version + uv.lock hash to reduce cache collisions.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/main.yaml Outdated
with:
path: .venv
key: venv-${{ hashFiles('**/uv.lock') }}
key: venv-${{ runner.os }}-${{ matrix.python-version }}-${{ hashFiles('**/uv.lock') }}
@iMicknl
iMicknl merged commit 17e3d2e into main Aug 16, 2026
8 checks passed
@iMicknl
iMicknl deleted the codex/add-pytest-ci branch August 16, 2026 18:39
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