Skip to content

Commit 79abe97

Browse files
committed
Adding the rest of CI stuff from PR #33.
1 parent b3051c0 commit 79abe97

File tree

1 file changed

+30
-36
lines changed

1 file changed

+30
-36
lines changed

.github/workflows/continuous-integration.yml

Lines changed: 30 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -45,19 +45,13 @@ jobs:
4545
cache: 'sbt'
4646

4747
- name: Build & Test
48-
run: sbt ++${{ matrix.scala }} testWithCoverage
48+
run: sbt ++${{ matrix.scala }} clean testWithCoverage
4949

50-
- name: Upload coverage report (Cobertura)
50+
- name: Upload coverage report (all)
5151
uses: actions/[email protected]
5252
with:
53-
name: cobertura.xml
54-
path: ${{github.workspace}}/target/scala-2.13/coverage-report/cobertura.xml
55-
56-
- name: Upload coverage report (HTML)
57-
uses: actions/[email protected]
58-
with:
59-
name: scoverage-report-html
60-
path: ${{github.workspace}}/target/scala-2.13/scoverage-report/
53+
name: coverage-report-${{ matrix.scala }}
54+
path: ${{github.workspace}}/target/scala-${{ matrix.scala }}/coverage-report
6155

6256
optional-build:
6357
name: Build (Optional)
@@ -90,35 +84,35 @@ jobs:
9084
cache: 'sbt'
9185

9286
- name: Perform Build / Test
93-
run: sbt ++${{ matrix.scala }} compile test
94-
95-
coverage:
96-
name: Coverage Report
97-
if: ${{ github.event.pull_request }}
98-
needs: [ build ]
99-
runs-on: ubuntu-latest
100-
steps:
101-
- uses: actions/download-artifact@v3
102-
with:
103-
name: cobertura.xml
104-
105-
- name: Analyzing coverage report
106-
uses: 5monkeys/cobertura-action@master
107-
with:
108-
path: cobertura.xml
109-
only_changed_files: true
110-
fail_below_threshold: true
111-
show_missing: true
112-
show_line: true
113-
show_branch: true
114-
show_class_names: true
115-
link_missing_lines: true
116-
minimum_coverage: 75
87+
run: sbt ++${{ matrix.scala }} clean compile test
88+
89+
# coverage:
90+
# name: Coverage Report
91+
# if: ${{ github.event.pull_request }}
92+
# needs: [ build ]
93+
# runs-on: ubuntu-latest
94+
# steps:
95+
# - uses: actions/download-artifact@v3
96+
# with:
97+
# name: cobertura.xml
98+
#
99+
# - name: Analyzing coverage report
100+
# uses: 5monkeys/cobertura-action@master
101+
# with:
102+
# path: cobertura.xml
103+
# only_changed_files: true
104+
# fail_below_threshold: true
105+
# show_missing: true
106+
# show_line: true
107+
# show_branch: true
108+
# show_class_names: true
109+
# link_missing_lines: true
110+
# minimum_coverage: 75
117111

118112
ready-to-merge:
119113
name: Ready to Merge
120114
if: ${{ github.event.pull_request }}
121-
needs: [ optional-build, coverage ]
115+
needs: [ optional-build ]
122116
runs-on: ubuntu-latest
123117
steps:
124-
- run: echo 'Ready to merge.'
118+
- run: echo 'Ready to merge.'

0 commit comments

Comments
 (0)