search

rdatsci / rtcl

R tools for the command line
65 stars 3 forks source link

new function: rtstchk #9

Closed surmann closed 9 years ago

surmann commented 9 years ago

Currently, rcheck aka devtools::check does not run the unit tests. They are only run by rtest aka devtools::test. Should we implement a function rtstchk which runs the unit test first and than the R CMD check?

jakob-r commented 9 years ago

:+1: +1

berndbischl commented 9 years ago

To be honest, I find it quite nice that the 2 are separate. But I also see no hurt in adding a separate command. Do a PR?

surmann commented 9 years ago

It is ok. But you have to know it. I discovered it while developing ecr. Yesterday it was a new info for some colleagues. If you assume rcheck do the tests, you can run into problems.

If you guys think it is helpful I implement the function.

surmann commented 9 years ago

Done in commit 28abf6616963c7d590af1d8e9995c10ec08f8172