Skip to content

Commit

Permalink
Include run & attempt number in check-r-package artifact names
Browse files Browse the repository at this point in the history
So you can run use the same OS & arch multiple times. This was
a problem because of a change in the upload-artifact action.
  • Loading branch information
gaborcsardi committed Feb 1, 2024
1 parent ab0a542 commit 071d49b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions check-r-package/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -54,13 +54,13 @@ runs:
if: failure() || inputs.upload-results != 'false'
uses: actions/upload-artifact@v4
with:
name: ${{ runner.os }}-r${{ matrix.config.r }}-results
name: ${{ runner.os }}-${{ runner.arch }}-r${{ matrix.config.r }}-${{ github.run_number }}-${{ github.run_attempt }}-results
path: ${{ steps.rcmdcheck.outputs.check-dir-path }}

- name: Upload snapshots
if: inputs.upload-snapshots != 'false'
uses: actions/upload-artifact@v4
with:
name: ${{ runner.os }}-${{ runner.arch }}-r${{ matrix.config.r }}-testthat-snapshots
name: ${{ runner.os }}-${{ runner.arch }}-r${{ matrix.config.r }}-${{ github.run_number }}-${{ github.run_attempt }}-testthat-snapshots
path: ${{ steps.rcmdcheck.outputs.check-dir-path }}/**/tests*/testthat/_snaps
if-no-files-found: ignore

0 comments on commit 071d49b

Please sign in to comment.