Skip to content

Commit

Permalink
Hotfix: change matrix.os to matrix.config.os so covtests run
Browse files Browse the repository at this point in the history
  • Loading branch information
nickp60 committed Apr 28, 2023
1 parent 96f19e7 commit 3467102
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/R-CMD-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ jobs:
with:
upload-snapshots: true
- name: Test coverage
if: matrix.os == 'ubuntu-latest'
if: matrix.config.os == 'ubuntu-latest'
run: |
covr::codecov(
quiet = FALSE,
Expand All @@ -58,14 +58,14 @@ jobs:
shell: Rscript {0}

- name: Show testthat output
if: matrix.os == 'ubuntu-latest'
if: matrix.config.os == 'ubuntu-latest'
run: |
## --------------------------------------------------------------------
find ${{ runner.temp }}/package -name 'testthat.Rout*' -exec cat '{}' \; || true
shell: bash

- name: Upload test results
if: failure() && matrix.os == 'ubuntu-latest'
if: failure() && matrix.config.os == 'ubuntu-latest'
uses: actions/upload-artifact@v3
with:
name: coverage-test-failures
Expand Down

0 comments on commit 3467102

Please sign in to comment.