Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Small suggestion: use tinytest instead of testthat, to be more in line with fastverse principles #443

Open
tony-aw opened this issue Aug 17, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@tony-aw
Copy link

tony-aw commented Aug 17, 2023

Hi,

Small suggestion:
To make the collapse R package more in line with fastverse (and tinyverse) principles, you might want to perhaps run your tests using the tinytest R package, instead of testthat. This is just a small suggestion though, nothing more.

Kind regards,

Tony

@SebKrantz
Copy link
Owner

SebKrantz commented Aug 18, 2023

Thanks, I agree it's a huge bloat to install on a server. I'll look into it as I find time. I like the integration of testthat with Rstudio, that's why I have kept it, and it doesn't really matter for CRAN as those servers have all packages preinstalled.

You can omit installing suggested packages by adding dependencies = "NA" to your YAML:

jobs:
  MyJob:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v3
      - uses: r-lib/actions/setup-r@v2
      - uses: r-lib/actions/setup-r-dependencies@v2
        with:
          dependencies: "NA"

@SebKrantz SebKrantz added the enhancement New feature or request label Feb 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants