Skip to content

Commit

Permalink
CI: Fix artefact uploads
Browse files Browse the repository at this point in the history
With the current version of the action, we must avoid overriding
artefacts in matrix builds.
  • Loading branch information
daniel-ac-martin committed Feb 14, 2024
1 parent 0764471 commit 1bd7b35
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
8 changes: 6 additions & 2 deletions .github/actions/test-code/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,10 @@ inputs:
description: Name of branch to store baseline on
required: false
default: master
node:
description: The version of Node.js to use.
required: false
default: 18
runs:
using: composite
steps:
Expand All @@ -16,11 +20,11 @@ runs:
- name: Upload test results
uses: actions/upload-artifact@master
with:
name: test-results
name: test-results-${{ inputs.node }}
path: .jest-results.json

- name: Upload coverage report
uses: actions/upload-artifact@master
with:
name: coverage-report
name: coverage-report-${{ inputs.node }}
path: coverage
2 changes: 2 additions & 0 deletions .github/workflows/change-assurance.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,8 @@ jobs:

- name: Run unit tests
uses: ./.github/actions/test-code
with:
node: ${{ matrix.node }}

libs:
name: Build libraries
Expand Down

1 comment on commit 1bd7b35

@github-actions
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.