Skip to content

Commit a94304a

Browse files
committed
return coveralls after resolve coverallsapp/coverage-reporter#180
Signed-off-by: Slach <[email protected]>
1 parent 8a9447a commit a94304a

File tree

1 file changed

+35
-38
lines changed

1 file changed

+35
-38
lines changed

.github/workflows/build.yaml

Lines changed: 35 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -55,14 +55,13 @@ jobs:
5555
make build/linux/amd64/clickhouse-backup-fips build/linux/arm64/clickhouse-backup-fips
5656
make build-race build-race-fips config test
5757
58-
# todo return when resolve https://github.com/coverallsapp/coverage-reporter/issues/180
59-
# - name: Report unittest coverage
60-
# uses: coverallsapp/github-action@v2
61-
# with:
62-
# file: _coverage_/coverage.out
63-
# parallel: true
64-
# format: golang
65-
# flag-name: unittest-${{ matrix.clickhouse }}
58+
- name: Report unittest coverage
59+
uses: coverallsapp/github-action@v2
60+
with:
61+
file: _coverage_/coverage.out
62+
parallel: true
63+
format: golang
64+
flag-name: unittest-${{ matrix.clickhouse }}
6665

6766
- name: Extract GCS credentials
6867
id: secrets
@@ -177,15 +176,14 @@ jobs:
177176
ls -la test/testflows/_coverage_
178177
go env
179178
go tool covdata textfmt -i test/testflows/_coverage_/ -o test/testflows/_coverage_/coverage.out
180-
# todo return when resolve https://github.com/coverallsapp/coverage-reporter/issues/180
181-
# - name: Report testflows coverage
182-
# uses: coverallsapp/github-action@v2
183-
# with:
184-
# base-path: ./
185-
# file: test/testflows/_coverage_/coverage.out
186-
# parallel: true
187-
# format: golang
188-
# flag-name: testflows-${{ matrix.clickhouse }}
179+
- name: Report testflows coverage
180+
uses: coverallsapp/github-action@v2
181+
with:
182+
base-path: ./
183+
file: test/testflows/_coverage_/coverage.out
184+
parallel: true
185+
format: golang
186+
flag-name: testflows-${{ matrix.clickhouse }}
189187
# todo possible failures https://github.com/actions/upload-artifact/issues/270
190188
- name: Upload testflows logs
191189
uses: actions/upload-artifact@v4
@@ -345,27 +343,26 @@ jobs:
345343
sudo chmod -Rv a+rw test/integration/_coverage_/
346344
ls -la test/integration/_coverage_
347345
go tool covdata textfmt -i test/integration/_coverage_/ -o test/integration/_coverage_/coverage.out
348-
# todo return when resolve https://github.com/coverallsapp/coverage-reporter/issues/180
349-
# - name: Report integration coverage
350-
# uses: coverallsapp/github-action@v2
351-
# with:
352-
# base-path: ./
353-
# file: test/integration/_coverage_/coverage.out
354-
# parallel: true
355-
# format: golang
356-
# flag-name: integration-${{ matrix.clickhouse }}
357-
# coverage:
358-
# needs:
359-
# - test
360-
# - testflows
361-
# name: coverage
362-
# runs-on: ubuntu-24.04
363-
# steps:
364-
# - name: Coveralls Finished
365-
# uses: coverallsapp/github-action@v2
366-
# with:
367-
# base-path: ./
368-
# parallel-finished: true
346+
- name: Report integration coverage
347+
uses: coverallsapp/github-action@v2
348+
with:
349+
base-path: ./
350+
file: test/integration/_coverage_/coverage.out
351+
parallel: true
352+
format: golang
353+
flag-name: integration-${{ matrix.clickhouse }}
354+
coverage:
355+
needs:
356+
- test
357+
- testflows
358+
name: coverage
359+
runs-on: ubuntu-24.04
360+
steps:
361+
- name: Coveralls Finished
362+
uses: coverallsapp/github-action@v2
363+
with:
364+
base-path: ./
365+
parallel-finished: true
369366
docker:
370367
needs:
371368
- test

0 commit comments

Comments
 (0)