Skip to content

Commit

Permalink
Fix PR handling on coverage report
Browse files Browse the repository at this point in the history
  • Loading branch information
mmicko committed Apr 15, 2024
1 parent 8543895 commit f7e58fd
Show file tree
Hide file tree
Showing 2 changed files with 26 additions and 5 deletions.
7 changes: 2 additions & 5 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,13 +13,10 @@ jobs:
run: make dev-install
- name: Run tests
run: make test
- name: Report
uses: 5monkeys/cobertura-action@master
- uses: actions/upload-artifact@v4
with:
name: coverage
path: coverage.xml
minimum_coverage: 90
skip_covered: false
fail_below_threshold: true

check:
runs-on: ubuntu-20.04
Expand Down
24 changes: 24 additions & 0 deletions .github/workflows/report.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
name: report
on:
workflow_run:
workflows: [ci]
types:
- completed

jobs:
test:
runs-on: ubuntu-20.04 # Ubuntu 20.04 LTS ships with the Python version we're currently targeting

steps:
- uses: actions/download-artifact@v4
with:
name: coverage
path: coverage.xml

- name: Report
uses: 5monkeys/cobertura-action@master
with:
path: coverage.xml
minimum_coverage: 90
skip_covered: false
fail_below_threshold: true

0 comments on commit f7e58fd

Please sign in to comment.