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) <noreply@anthropic.com>
This commit is contained in:
Mathew Sir Guest the best 2026-05-31 00:04:34 -06:00
parent 6931e91391
commit 84e7639bf1
2 changed files with 2 additions and 11 deletions

@ -17,14 +17,5 @@ jobs:
- name: Install uv - name: Install uv
uses: astral-sh/setup-uv@v5 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 - name: Run smoke tests
run: uv run pytest -v run: uv run --python ${{ matrix.python-version }} --with . --with pytest pytest -v

@ -36,7 +36,7 @@ Smoke tests live in `tests/` (import, CLI `--help`, and settings checks) and run
## Code Style ## Code Style
- **Black** formatter: 120 char line length, target Python 3.13 - **Black** formatter: 120 char line length, targets Python 3.83.13
- **isort**: profile black, multi_line_output=3, trailing commas, force_grid_wrap=3 - **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 - **flake8**: 120 char max, ignores E121/E123/E126/E226/E24/E704/W605