Skip to content

Commit

Permalink
[CI] Reworked coverage & artifact generation
Browse files Browse the repository at this point in the history
- The artifacts' names are now prepended with "RaZ-", and the job ID is replaced by the commit's SHA

- Removed all files in extern/ for coverage processing, as they're not part of the engine and shouldn't be counted in the coverage percentage

- Removed Coveralls' action's flag-name attribute, which doesn't really make sense as it is and is optional anyway
  • Loading branch information
Razakhel committed Jan 24, 2024
1 parent db53b4a commit 3ce733d
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions .github/workflows/RaZ.yml
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ jobs:
working-directory: ${{ runner.workspace }}/build-${{ matrix.compiler.c }}
run: |
lcov -d ./CMakeFiles/RaZ.dir/src/ -c -o coverage.info &&
lcov -r coverage.info '/usr/*' -o coverage.info &&
lcov -r coverage.info '/usr/*' '${{ github.workspace }}/extern/*' -o coverage.info &&
lcov -l coverage.info
# Processing code coverage; only done with GCC in Debug with all features enabled
Expand All @@ -115,7 +115,6 @@ jobs:
uses: coverallsapp/github-action@master
with:
github-token: ${{ secrets.GITHUB_TOKEN }}
flag-name: github-actions-${{ github.run_id }}
path-to-lcov: ${{ runner.workspace }}/build-${{ matrix.compiler.c }}/coverage.info

# Installing RaZ in the build directory; default is /usr/local
Expand All @@ -127,7 +126,7 @@ jobs:
if: matrix.use_audio && matrix.use_window
uses: actions/upload-artifact@v3
with:
name: linux-${{ matrix.compiler.c }}-${{ matrix.build_type }}-${{ github.run_id }}
name: RaZ-linux-${{ matrix.compiler.c }}-${{ matrix.build_type }}-${{ github.sha }}
path: |
${{ runner.workspace }}/build-${{ matrix.compiler.c }}/include
${{ runner.workspace }}/build-${{ matrix.compiler.c }}/lib
Expand Down Expand Up @@ -215,7 +214,7 @@ jobs:
if: matrix.use_audio && matrix.use_window
uses: actions/upload-artifact@v3
with:
name: windows-${{ matrix.compiler.c }}-${{ matrix.build_type }}-${{ github.run_id }}
name: RaZ-windows-${{ matrix.compiler.c }}-${{ matrix.build_type }}-${{ github.sha }}
path: |
C:/RaZ
Expand Down Expand Up @@ -292,7 +291,7 @@ jobs:
if: matrix.use_audio && matrix.use_window
uses: actions/upload-artifact@v3
with:
name: macos-${{ matrix.compiler.c }}-${{ matrix.build_type }}-${{ github.run_id }}
name: RaZ-macos-${{ matrix.compiler.c }}-${{ matrix.build_type }}-${{ github.sha }}
path: |
${{ runner.workspace }}/build-${{ matrix.compiler.c }}/include
${{ runner.workspace }}/build-${{ matrix.compiler.c }}/lib
Expand Down

0 comments on commit 3ce733d

Please sign in to comment.