Skip to content

Commit

Permalink
upload entire coverage report directory, and remove coverage threshol…
Browse files Browse the repository at this point in the history
…d check pending cequence-io#24
  • Loading branch information
phelps-sg committed Jun 18, 2023
1 parent 0cf5fa8 commit 614ac85
Showing 1 changed file with 25 additions and 31 deletions.
56 changes: 25 additions & 31 deletions .github/workflows/continuous-integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,17 +47,11 @@ jobs:
- name: Build & Test
run: sbt ++${{ matrix.scala }} clean testWithCoverage

- name: Upload coverage report (Cobertura)
- name: Upload coverage report (all)
uses: actions/[email protected]
with:
name: cobertura.xml
path: ${{github.workspace}}/target/scala-${{ matrix.scala }}/coverage-report/cobertura.xml

- name: Upload coverage report (HTML)
uses: actions/[email protected]
with:
name: scoverage-report-html
path: ${{github.workspace}}/target/scala-${{ matrix.scala }}/scoverage-report/
name: coverage-report-${{ matrix.scala }}
path: ${{github.workspace}}/target/scala-${{ matrix.scala }}/coverage-report

optional-build:
name: Build (Optional)
Expand Down Expand Up @@ -92,28 +86,28 @@ jobs:
- name: Perform Build / Test
run: sbt ++${{ matrix.scala }} clean compile test

coverage:
name: Coverage Report
if: ${{ github.event.pull_request }}
needs: [ build ]
runs-on: ubuntu-latest
steps:
- uses: actions/download-artifact@v3
with:
name: cobertura.xml

- name: Analyzing coverage report
uses: 5monkeys/cobertura-action@master
with:
path: cobertura.xml
only_changed_files: true
fail_below_threshold: true
show_missing: true
show_line: true
show_branch: true
show_class_names: true
link_missing_lines: true
minimum_coverage: 75
# coverage:
# name: Coverage Report
# if: ${{ github.event.pull_request }}
# needs: [ build ]
# runs-on: ubuntu-latest
# steps:
# - uses: actions/download-artifact@v3
# with:
# name: cobertura.xml
#
# - name: Analyzing coverage report
# uses: 5monkeys/cobertura-action@master
# with:
# path: cobertura.xml
# only_changed_files: true
# fail_below_threshold: true
# show_missing: true
# show_line: true
# show_branch: true
# show_class_names: true
# link_missing_lines: true
# minimum_coverage: 75

ready-to-merge:
name: Ready to Merge
Expand Down

0 comments on commit 614ac85

Please sign in to comment.