Skip to content

Commit

Permalink
Merge pull request #420 from maxekman/fix/ci-use-multiple-artifacts
Browse files Browse the repository at this point in the history
fix(ci): use multiple artifacts for code coverage
  • Loading branch information
maxekman authored Jan 5, 2025
2 parents 877bcc0 + 8bd29cd commit b211160
Showing 1 changed file with 9 additions and 4 deletions.
13 changes: 9 additions & 4 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ jobs:
- name: Store coverage
uses: actions/upload-artifact@v4
with:
name: coverage
name: unit_coverage
path: unit.coverprofile

integration:
Expand All @@ -53,7 +53,7 @@ jobs:
- name: Store coverage
uses: actions/upload-artifact@v4
with:
name: coverage
name: integration_coverage
path: integration.coverprofile

finish:
Expand All @@ -68,10 +68,15 @@ jobs:
with:
go-version: 1.16

- name: Load coverage
- name: Load unit coverage
uses: actions/download-artifact@v4
with:
name: coverage
name: unit_coverage

- name: Load integration coverage
uses: actions/download-artifact@v4
with:
name: integration_coverage

- name: Merge coverage
shell: bash
Expand Down

0 comments on commit b211160

Please sign in to comment.