From 84e7639bf139730aa794c44de55a2a7fde4b740a Mon Sep 17 00:00:00 2001 From: Mathew Sir Guest the best Date: Sun, 31 May 2026 00:04:34 -0600 Subject: [PATCH] ci: run pytest via uv run so the matrix interpreter is honored Address code review: the previous uv venv + uv pip + uv run sequence could run pytest in a different environment than the matrix Python. Use a single uv run --python ... --with . --with pytest invocation (mirrors noxfile). Also correct the stale Black target-version note in CLAUDE.md. Co-Authored-By: Claude Opus 4.8 (1M context) --- .gitea/workflows/test.yml | 11 +---------- CLAUDE.md | 2 +- 2 files changed, 2 insertions(+), 11 deletions(-) diff --git a/.gitea/workflows/test.yml b/.gitea/workflows/test.yml index 298aca2..4292afb 100644 --- a/.gitea/workflows/test.yml +++ b/.gitea/workflows/test.yml @@ -17,14 +17,5 @@ jobs: - name: Install uv uses: astral-sh/setup-uv@v5 - - name: Install Python ${{ matrix.python-version }} - run: uv python install ${{ matrix.python-version }} - - - name: Create venv - run: uv venv --python ${{ matrix.python-version }} - - - name: Install project and pytest - run: uv pip install . pytest - - name: Run smoke tests - run: uv run pytest -v + run: uv run --python ${{ matrix.python-version }} --with . --with pytest pytest -v diff --git a/CLAUDE.md b/CLAUDE.md index 098db49..edd1a90 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -36,7 +36,7 @@ Smoke tests live in `tests/` (import, CLI `--help`, and settings checks) and run ## Code Style -- **Black** formatter: 120 char line length, target Python 3.13 +- **Black** formatter: 120 char line length, targets Python 3.8–3.13 - **isort**: profile black, multi_line_output=3, trailing commas, force_grid_wrap=3 - **flake8**: 120 char max, ignores E121/E123/E126/E226/E24/E704/W605