Skip to content

Commit

Permalink
WIP
Browse files Browse the repository at this point in the history
  • Loading branch information
thomass-dev committed Dec 27, 2024
1 parent 07d5676 commit 58d9cfd
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 11 deletions.
20 changes: 11 additions & 9 deletions .github/workflows/backend.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@ jobs:
matrix:
os: ["ubuntu-latest", "windows-latest"]
python: ["3.9", "3.10", "3.11", "3.12"]
coverage: [false]
include:
- os: "ubuntu-latest"
python: "3.12"
Expand All @@ -64,19 +63,22 @@ jobs:
working-directory: skore/
run: wheel=(dist/*.whl); python -m pip install "${wheel}[test]"

- name: Test with/without coverage
- name: Test without coverage
if: ${{ ! matrix.coverage }}
timeout-minutes: 10
working-directory: skore/
run: python -m pytest -n auto --no-cov src/ tests/

- name: Test with coverage
if: ${{ matrix.coverage }}
timeout-minutes: 10
working-directory: skore/
run: |
if [ ${{ matrix.coverage }} = false ] ; then
python -m pytest -n auto --no-cov src/ tests/
else
mkdir coverage
python -m pytest -n auto --junitxml=coverage/coverage.xml --cov=skore src/ tests/ | tee coverage/coverage.txt
fi
mkdir coverage
python -m pytest -n auto --junitxml=coverage/coverage.xml --cov=skore src/ tests/ | tee coverage/coverage.txt
- name: Upload coverage reports
if: matrix.coverage == 'true'
if: ${{ matrix.coverage }}
uses: actions/upload-artifact@v4
with:
name: backend-coverage
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
# IS
# DOCUMENTATION

name: CI
name: ci

on:
pull_request:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/pr-display-backend-coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ name: display backend coverage in PR

on:
workflow_run:
workflows: [backend]
workflows: [Lint PR title]
types: [completed]

concurrency:
Expand Down

0 comments on commit 58d9cfd

Please sign in to comment.