Skip to content

Commit

Permalink
Remove unneeded deps from suggests + add test coverage workflow.
Browse files Browse the repository at this point in the history
  • Loading branch information
olivroy committed Feb 21, 2024
1 parent fd1d83e commit 44bc603
Show file tree
Hide file tree
Showing 2 changed files with 51 additions and 4 deletions.
50 changes: 50 additions & 0 deletions .github/workflows/test-coverage.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,50 @@
# Workflow derived from https://github.com/r-lib/actions/tree/v2/examples
# Need help debugging build failures? Start at https://github.com/r-lib/actions#where-to-find-help
on:
push:
branches: [main, master]
pull_request:
branches: [main, master]

name: test-coverage

jobs:
test-coverage:
runs-on: ubuntu-latest
env:
GITHUB_PAT: ${{ secrets.GITHUB_TOKEN }}

steps:
- uses: actions/checkout@v4

- uses: r-lib/actions/setup-r@v2
with:
use-public-rspm: true

- uses: r-lib/actions/setup-r-dependencies@v2
with:
extra-packages: any::covr
needs: coverage

- name: Test coverage
run: |
covr::codecov(
quiet = FALSE,
clean = FALSE,
install_path = file.path(normalizePath(Sys.getenv("RUNNER_TEMP"), winslash = "/"), "package")
)
shell: Rscript {0}

- name: Show testthat output
if: always()
run: |
## --------------------------------------------------------------------
find ${{ runner.temp }}/package -name 'testthat.Rout*' -exec cat '{}' \; || true
shell: bash

- name: Upload test results
if: failure()
uses: actions/upload-artifact@v4
with:
name: coverage-test-failures
path: ${{ runner.temp }}/package
5 changes: 1 addition & 4 deletions DESCRIPTION
Original file line number Diff line number Diff line change
Expand Up @@ -52,17 +52,14 @@ Imports:
tsibble (>= 0.8.2),
vctrs
Suggests:
markdown,
covr,
gapminder,
gghighlight (>= 0.1.0),
knitr (>= 1.23),
lme4,
modelr,
readr (>= 1.3.1),
rmarkdown (>= 1.14),
spelling (>= 2.1),
testthat (>= 2.1.0),
testthat (>= 3.0.0),
tsibbledata,
vdiffr (>= 0.3.1)
VignetteBuilder:
Expand Down

0 comments on commit 44bc603

Please sign in to comment.