Skip to content

Commit

Permalink
Use unique artifact names
Browse files Browse the repository at this point in the history
Signed-off-by: Fabrice Normandin <[email protected]>
  • Loading branch information
lebrice committed May 15, 2024
1 parent 393cf70 commit ed1714f
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
10 changes: 6 additions & 4 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ jobs:
- name: Store coverage report as an artifact
uses: actions/upload-artifact@v4
with:
name: coverage-reports
name: coverage-reports-unit-${{ matrix.platform }}-${{ matrix.python-version }}
path: ./coverage.xml

mock-slurm-integration-tests:
Expand Down Expand Up @@ -150,7 +150,7 @@ jobs:
- name: Store coverage report as an artifact
uses: actions/upload-artifact@v4
with:
name: coverage-reports
name: coverage-reports-mock-${{ matrix.platform }}-${{ matrix.python-version }}
path: ./coverage.xml

real-slurm-integration-tests:
Expand Down Expand Up @@ -182,9 +182,11 @@ jobs:
- name: Download artifacts
uses: actions/download-artifact@v4
with:
name: coverage-reports
pattern: coverage-reports-*
merge-multiple: false
# download all the artifacts in this directory (each .coverage.xml will be in a subdirectory)
# Next step if this doesn't work would be to give the coverage files a unique name and use merge-multiple: true
path: coverage_reports
# merge-multiple: false
- name: Upload coverage reports to Codecov
uses: codecov/codecov-action@v4
with:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -53,5 +53,5 @@ jobs:
- name: Store coverage report as an artifact
uses: actions/upload-artifact@v4
with:
name: coverage-reports
name: coverage-reports-integration-${{ inputs.cluster }}-${{ inputs.python-version }}
path: ./coverage.xml

0 comments on commit ed1714f

Please sign in to comment.