Skip to content

Commit 7210028

Browse files
ci: run coverage only when required
Before this patch, we ran coverage tests even when results weren't used in coveralls. Running coverage tests is significantly more expensive (up to 5 times in length).
1 parent a92a836 commit 7210028

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

.github/workflows/test_on_push.yaml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -74,10 +74,11 @@ jobs:
7474
env:
7575
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
7676

77-
- name: Run tests and code coverage analysis
78-
run: make -C build coverage
77+
- name: Run tests
78+
if: matrix.coveralls != true # See https://github.com/actions/runner/issues/1173
79+
run: make -C build luatest-no-coverage
7980

80-
- name: Send code coverage to coveralls.io
81+
- name: Run tests with coverage and send results to coveralls.io
8182
run: make -C build coveralls
8283
if: ${{ matrix.coveralls }}
8384

@@ -153,5 +154,5 @@ jobs:
153154

154155
- run: cmake -S . -B build
155156

156-
- name: Run tests and code coverage analysis
157-
run: make -C build coverage
157+
- name: Run tests
158+
run: make -C build luatest-no-coverage

0 commit comments

Comments
 (0)