mirror of
https://github.com/mvt-project/mvt.git
synced 2026-09-11 12:18:57 +02:00
Donncha Ó Cearbhaill
03012d2c2d
The coverage comment never posted: the per-file table with a link on every file and missing line range exceeds GitHub's 65536-character comment limit, so PRs got only a badge. Every matrix job also raced to post the same comment, and the action was unpinned at @main. Post from one job only, limit the table to files changed in the PR, drop per-line links, and pin the action. Write the full coverage table to the job summary as well, which also works for PRs from forks where the token is read-only. Set UV_PYTHON from the matrix. Without it, .python-version pins 3.10 and `uv run` rebuilt the venv with 3.10 after `uv sync --python X`, so all five matrix jobs were testing on Python 3.10.
57 lines
1.8 KiB
YAML
57 lines
1.8 KiB
YAML
name: Tests
|
|
on:
|
|
push:
|
|
branches: [ main ]
|
|
pull_request:
|
|
branches: [ main ]
|
|
|
|
permissions:
|
|
contents: read
|
|
pull-requests: write # coverage comment
|
|
|
|
jobs:
|
|
build:
|
|
name: Run Python Tests
|
|
runs-on: ubuntu-latest
|
|
strategy:
|
|
fail-fast: false
|
|
matrix:
|
|
python-version: ['3.10', '3.11', '3.12', '3.13', '3.14']
|
|
env:
|
|
# Takes precedence over .python-version, which otherwise makes `uv run`
|
|
# rebuild the venv with 3.10 and test every matrix entry on 3.10.
|
|
UV_PYTHON: ${{ matrix.python-version }}
|
|
|
|
steps:
|
|
- uses: actions/checkout@v7
|
|
- name: Install uv
|
|
uses: astral-sh/setup-uv@v10.0.0
|
|
with:
|
|
enable-cache: true
|
|
- name: Set up Python ${{ matrix.python-version }}
|
|
uses: actions/setup-python@v7
|
|
with:
|
|
python-version: ${{ matrix.python-version }}
|
|
- name: Install Python dependencies
|
|
run: |
|
|
uv sync --locked --group dev
|
|
- name: Test with pytest
|
|
run: |
|
|
set -o pipefail
|
|
make test-ci | tee pytest-coverage.txt
|
|
- name: Coverage job summary
|
|
run: uv run coverage report --format=markdown --show-missing --skip-covered >> "$GITHUB_STEP_SUMMARY"
|
|
|
|
- name: Pytest coverage comment
|
|
# One comment per PR, not one per matrix entry. PRs from forks get a
|
|
# read-only token and can't post; the job summary above still works.
|
|
if: github.event_name == 'pull_request' && matrix.python-version == '3.13'
|
|
continue-on-error: true
|
|
uses: MishaKav/pytest-coverage-comment@v1.12.2
|
|
with:
|
|
pytest-coverage-path: ./pytest-coverage.txt
|
|
junitxml-path: ./pytest.xml
|
|
# The full table with per-line links exceeds GitHub's 65536-char comment limit.
|
|
report-only-changed-files: true
|
|
remove-links-to-lines: true
|