Skip to content

Commit a9bacc9

Browse files
committed
Makefile.maker.yaml: remove Coveralls
This is required because of <sapcc/go-makefile-maker#351>.
1 parent 085431e commit a9bacc9

File tree

2 files changed

+23
-8
lines changed

2 files changed

+23
-8
lines changed

.github/workflows/ci.yaml

Lines changed: 23 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,24 @@ jobs:
3333
with:
3434
check-latest: true
3535
go-version: 1.24.6
36+
code_coverage:
37+
name: Code coverage report
38+
if: github.event_name == 'pull_request'
39+
needs:
40+
- test
41+
runs-on: ubuntu-latest
42+
steps:
43+
- name: Check out code
44+
uses: actions/checkout@v5
45+
- name: Post coverage report
46+
uses: fgrosse/go-coverage-report@v1.2.0
47+
with:
48+
coverage-artifact-name: code-coverage
49+
coverage-file-name: cover.out
50+
permissions:
51+
actions: read
52+
contents: read
53+
pull-requests: write
3654
test:
3755
name: Test
3856
needs:
@@ -52,10 +70,8 @@ jobs:
5270
sudo apt-get install -y --no-install-recommends postgresql-17
5371
export PATH=/usr/lib/postgresql/17/bin:$PATH
5472
make build/cover.out
55-
- name: Upload coverage report to Coveralls
56-
env:
57-
COVERALLS_TOKEN: ${{ secrets.GITHUB_TOKEN }}
58-
GIT_BRANCH: ${{ github.head_ref }}
59-
run: |
60-
go install github.com/mattn/goveralls@latest
61-
goveralls -service=github -coverprofile=build/cover.out
73+
- name: Archive code coverage results
74+
uses: actions/upload-artifact@v4
75+
with:
76+
name: code-coverage
77+
path: build/cover.out

Makefile.maker.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ metadata:
66
githubWorkflow:
77
ci:
88
enabled: true
9-
coveralls: true
109

1110
golang:
1211
autoupdateableDeps: ^github.com/(?:sapcc|sap-cloud-infrastructure)/

0 commit comments

Comments
 (0)