Skip to content

Commit 3f92fcb

Browse files
committed
Coverage-reporting CI job: don't run lcov twice
cmake --target coverage already takes care of running lcov. We ended up doing exactly the same work twice, just logging to different output files. This should save about 90 seconds of CI job execution time.
1 parent cc9d04e commit 3f92fcb

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

.github/workflows/pull-request-checks.yaml

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -699,15 +699,11 @@ jobs:
699699
run: cmake --build build -- -j2
700700
- name: Print ccache stats
701701
run: ccache -s
702-
- name: Run CTest
702+
- name: Run CTest and collect coverage statistics
703703
run: cmake --build build --target coverage -- -j2
704-
- name: Collect coverage statistics
705-
run: |
706-
lcov --capture --directory build --output-file lcov.info
707-
lcov --remove lcov.info '/usr/*' --output-file lcov.info
708704
- name: Upload coverage statistics to Codecov
709705
uses: codecov/codecov-action@v2
710706
with:
711-
files: ./lcov.info
707+
files: build/html/coverage.info
712708
fail_ci_if_error: true
713709
verbose: true

0 commit comments

Comments
 (0)