Skip to content

Commit c4a9409

Browse files
committed
Fix testing + coverage
* Drop `py312` to combat #16 * Install `--editable`, so that Python uses the correct source files * Ignore `tests/input/**/*.py` from coverage - they are not ran properly * Again improve on artifact naming Signed-off-by: Stavros Ntentos <[email protected]>
1 parent 55e4218 commit c4a9409

File tree

3 files changed

+8
-3
lines changed

3 files changed

+8
-3
lines changed

.codecov.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
ignore:
2+
- tests/input/**/*.py

.github/workflows/run-tests.yaml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,9 @@ jobs:
4141
steps:
4242
- uses: actions/checkout@v3
4343

44+
- name: Inject slug/short variables
45+
uses: rlespinasse/github-slug-action@v4
46+
4447
- name: Set up Python ${{ matrix.python-version }}
4548
uses: actions/setup-python@v4
4649
with:
@@ -71,5 +74,5 @@ jobs:
7174
uses: actions/upload-artifact@v3
7275
if: ${{ !cancelled() }}
7376
with:
74-
name: test-artifacts_${{ matrix.os }}_${{ matrix.python-version }}
77+
name: ${{ env.GITHUB_REPOSITORY_NAME_PART }}_test-artifacts_${{ github.event_name }}_${{ github.event.pull_request.number || env.GITHUB_SHA_SHORT }}_${{ matrix.os }}_py${{ matrix.python-version }}
7578
path: test_artifacts/

tox.ini

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
[tox]
2-
envlist = py36,py37,py38,py39,py310,py311,py312
2+
envlist = py36,py37,py38,py39,py310,py311
33
skipsdist = True
44
passenv =
55
FORCE_COLOR
@@ -9,5 +9,5 @@ deps =
99
pytest
1010
pytest-cov
1111
commands =
12-
pip install --upgrade .
12+
pip install --upgrade --editable .
1313
pytest --cov --cov-append {env:PYTEST_CI_ARGS:} {tty:--color=yes} {posargs:tests}

0 commit comments

Comments
 (0)