From 525db87185d68e722c32658f0bf045bae927ead5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Emmanuel=20Lepage=20Vall=C3=A9e?= Date: Sun, 25 Aug 2024 18:23:14 +0200 Subject: [PATCH] Move the C coverage to the CodeCov GitHub Action Same as Last PR. Now I expect the Lua coverage to work and the C coverage. I did it in 2 PRs to make sure the delta works. I also added a `-j8` in the example test step because it is I/O bound and it has enough ram. This should speed-up the CI (maybe?). --- .github/workflows/main.yml | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml index c04d06987..ee944b2c1 100644 --- a/.github/workflows/main.yml +++ b/.github/workflows/main.yml @@ -240,10 +240,6 @@ jobs: if: matrix.coverage run: sudo -H luarocks install cluacov - - name: Install codecov.io uploader - if: matrix.coverage == 'codecov' - run: wget -O /tmp/codecov-bash https://codecov.io/bash - # Check out repository to ${{ github.workspace }} # Automatically picks the current branch/PR - uses: actions/checkout@v2 @@ -308,7 +304,7 @@ jobs: run: cd "${{ github.workspace }}/build" && make check-unit - name: Run examples tests - run: cd "${{ github.workspace }}/build" && make check-examples + run: cd "${{ github.workspace }}/build" && make check-examples -j8 - name: Run requires tests if: matrix.coverage @@ -330,7 +326,7 @@ jobs: env: CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }} - - name: Upload C code coverage report + - name: Generate C coverage report if: matrix.coverage == 'codecov' run: | # Report coverage for each .gcno file separately. @@ -349,9 +345,13 @@ jobs: # with Code Climate. find "gcov.$i" -maxdepth 1 -type f -name '#usr#*.gcov' -delete done - - # Upload to Codecov. - bash /tmp/codecov-bash -X gcov -X coveragepy -F gcov + - name: Upload C code coverage report + if: matrix.coverage == 'codecov' + uses: codecov/codecov-action@v3 + with: + files: "${{ github.workspace }}/gcov" + env: + CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }} # `check-qa` is the only test that doesn't get a coverage report, so it has to run after all of that. - name: Run qa tests