Skip to content

Commit

Permalink
Fix artifact upload
Browse files Browse the repository at this point in the history
Since version 4 uploading multiple artifacts with the same name is not supported by actions/upload-artifact anymore.

See https://github.com/actions/upload-artifact/blob/cf8714cfeaba5687a442b9bcb85b29e23f468dfa/docs/MIGRATION.md for details.
  • Loading branch information
just95 authored Dec 26, 2023
1 parent 4bdaa15 commit 9e01b5e
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions .github/workflows/main.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ jobs:
- name: Upload Test Coverage
uses: actions/upload-artifact@v4
with:
name: coverage
name: coverage-vm-test-${ matrix.dart-version }
path: coverage/

# Runs the unit tests of the package with `dart test` in Node.js.
Expand Down Expand Up @@ -127,7 +127,7 @@ jobs:
- name: Upload Test Coverage
uses: actions/upload-artifact@v4
with:
name: coverage
name: coverage-js-test-${ matrix.node-version }
path: coverage/

# Runs the unit tests of the package with `dart test` in Chrome.
Expand All @@ -150,7 +150,7 @@ jobs:
- name: Upload Test Coverage
uses: actions/upload-artifact@v4
with:
name: coverage
name: coverage-chrome-test
path: coverage/

# Combines the test coverage information of all other jobs into a `lcov.info`
Expand All @@ -168,8 +168,9 @@ jobs:
- name: Download Coverage
uses: actions/download-artifact@v4
with:
name: coverage
pattern: coverage-*
path: coverage/
merge-multiple: true
- name: Use Dart SDK
uses: dart-lang/[email protected]
- name: Print Dart SDK Version
Expand Down

0 comments on commit 9e01b5e

Please sign in to comment.