estimatr icon indicating copy to clipboard operation
estimatr copied to clipboard

Consider adding skips for tests relying on suggested packages

Open michaelquinn32 opened this issue 3 years ago • 1 comments

Hi!

I hope this finds you well.

Several tests rely on randomizr, a suggested package. For cases where this package might not be installed, it would be nice if these tests cases were skipped. Testthat has a tool for this: skip_if_not_installed().

Would you be interested in a PR?

Thanks!

michaelquinn32 avatar Dec 06 '22 02:12 michaelquinn32

We would most def welcome a PR, although I am not sure if there are plans to cut a new release this school year. @acoppock ?

nfultz avatar Dec 06 '22 15:12 nfultz