brodieG / unitizer

Easy R Unit Tests
Other
39 stars 6 forks source link

Update arguments for devtools 2.0.0 #255

Closed jimhester closed 5 years ago

jimhester commented 5 years ago

devtools 2.0.0 removes the local argument in favor of build. The previous local = FALSE is approximately equal to build = TRUE.

This PR simply makes this change, which fixes the unitizer tests with devtools 2.0.0.

I plan to submit devtools to CRAN within the next week or so.

brodieG commented 5 years ago

Thanks for the heads up / PR. I'll review next week and prepare to re-submit to CRAN once devtools is update.

brodieG commented 5 years ago

So this pushed me to look back as to why I was using devtools::install, and it turns out that it was out of lazyness as I was having some issues with install.packages and install just worked. So I figured out what was wrong with install.packages and removed the suggests dependency. Thanks for taking the time to put in a PR, and sorry I did not end up using it. In the future feel free to just notify me without going through the trouble of submitting a PR.