Skip to content

Commit

Permalink
To fix github action warnings, upload-artifact version upgraded and d…
Browse files Browse the repository at this point in the history
…ifference plots are copied only on failure, which means they would be generated and available to copy
  • Loading branch information
scrasmussen committed Jul 20, 2024
1 parent e793599 commit f1bbb3b
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/test-pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -203,14 +203,14 @@ jobs:
cp -r $GITHUB_WORKSPACE/candidate/build/Run/output_${{ matrix.configuration }}/* $GITHUB_WORKSPACE/test_report/
- name: Attach diff plots to PR
if: ${{ always() }}
if: ${{ failure() }}
run: |
cd $GITHUB_WORKSPACE/candidate/tests/local/utils
bash attach_all_plots.bash $(jq --raw-output .pull_request.number "$GITHUB_EVENT_PATH") ${{ matrix.configuration }}
- name: Archive test results to GitHub
if: ${{ always() }}
uses: actions/upload-artifact@v3
if: ${{ failure() }}
uses: actions/upload-artifact@v4
with:
name: test-reports
path: |
Expand Down

0 comments on commit f1bbb3b

Please sign in to comment.