Skip to content

Commit

Permalink
PR: Fix the v3 -> v4 upload/download merging
Browse files Browse the repository at this point in the history
  • Loading branch information
shahzadlone committed Jun 27, 2024
1 parent ad40cac commit caded6b
Showing 1 changed file with 15 additions and 1 deletion.
16 changes: 15 additions & 1 deletion .github/workflows/test-and-upload-coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,17 @@ jobs:
if: ${{ !matrix.detect-changes }}
uses: actions/upload-artifact@v4
with:
name: ${{ matrix.os }}_${{ matrix.client-type }}_${{ matrix.database-type }}_${{ matrix.mutation-type }}
# Make sure the name is always unique per job as artifacts are now immutable.
# Note Issue: https://github.com/actions/upload-artifact/issues/478
# Solve: https://github.com/actions/upload-artifact/issues/478#issuecomment-1885470013
name: "coverage\
_${{ matrix.os }}\
_${{ matrix.client-type }}\
_${{ matrix.database-type }}\
_${{ matrix.mutation-type }}\
_${{ matrix.lens-type }}\
_${{ matrix.database-encryption }}\
"
path: coverage.txt
if-no-files-found: error
retention-days: 7
Expand All @@ -191,13 +201,17 @@ jobs:
- name: Download coverage reports
uses: actions/download-artifact@v4
with:
pattern: coverage_*
# Note: https://github.com/actions/download-artifact/blob/main/docs/MIGRATION.md
merge-multiple: false
path: coverage_reports

- name: Upload coverage to Codecov
uses: codecov/codecov-action@v4
with:
token: ${{ secrets.CODECOV_TOKEN }}
name: defradb-codecov
files: coverage_reports/**/*.txt
flags: all-tests
os: 'linux'
fail_ci_if_error: true
Expand Down

0 comments on commit caded6b

Please sign in to comment.