Skip to content

Commit d37460e

Browse files
authored
Merge pull request #170 from JSkimming/build-simpify
Simplify the coverage build to remove the matrix
2 parents 260bc9d + 50d40af commit d37460e

File tree

1 file changed

+7
-9
lines changed

1 file changed

+7
-9
lines changed

.github/workflows/ci.yaml

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ jobs:
3131
- uses: actions/checkout@v3
3232

3333
- name: Setup .NET 6.0
34-
uses: actions/setup-dotnet@v3.0.0
34+
uses: actions/setup-dotnet@v3
3535
if: matrix.framework == 'net6.0'
3636
with:
3737
dotnet-version: 6.0.x
@@ -42,35 +42,33 @@ jobs:
4242
code-coverage:
4343
runs-on: ubuntu-latest
4444
name: Code coverage
45-
strategy:
46-
matrix:
47-
framework:
48-
- net6.0
45+
env:
46+
framework: net6.0
4947

5048
steps:
5149

5250
- uses: actions/checkout@v3
5351

5452
- name: Setup .NET 6.0
55-
uses: actions/setup-dotnet@v3.0.0
53+
uses: actions/setup-dotnet@v3
5654
with:
5755
dotnet-version: 6.0.x
5856

5957
- name: Test Coverage
60-
run: ./coverage.sh ${{ matrix.framework }} Release
58+
run: ./coverage.sh ${{ env.framework }} Release
6159

6260
- name: Upload coverage to Codecov
6361
uses: codecov/codecov-action@v3
6462
with:
6563
token: ${{ secrets.CODECOV_TOKEN }}
66-
file: ./test/TestResults/output/coverage.${{ matrix.framework }}.cobertura.xml
64+
file: ./test/TestResults/output/coverage.${{ env.framework }}.cobertura.xml
6765
fail_ci_if_error: true
6866

6967
- name: Upload coverage to Coveralls
7068
uses: coverallsapp/github-action@master
7169
with:
7270
github-token: ${{ secrets.GITHUB_TOKEN }}
73-
path-to-lcov: ./test/TestResults/output/coverage.${{ matrix.framework }}.info
71+
path-to-lcov: ./test/TestResults/output/coverage.${{ env.framework }}.info
7472

7573
- name: Shellcheck
7674
run: >-

0 commit comments

Comments
 (0)