Why not use devtools::check() instead of R CMD check? #132
Comments
lcolladotor
added a commit
to leekgroup/derfinderHelper
that referenced
this issue
Sep 17, 2014
…#132 Also now dumping the timings info (testing proper syntax for this) and system info as documented in https://github.com/craigcitro/r-travis/wiki and from looking at travis-tool.sh
lcolladotor
added a commit
to leekgroup/derfinderHelper
that referenced
this issue
Jul 26, 2017
…#132 Also now dumping the timings info (testing proper syntax for this) and system info as documented in https://github.com/craigcitro/r-travis/wiki and from looking at travis-tool.sh From: lcolladotor <lcollado@jhsph.edu> git-svn-id: file:///home/git/hedgehog.fhcrc.org/bioconductor/trunk/madman/Rpacks/derfinderHelper@105209 bc3139a8-67e5-0310-9ffc-ced21a209358
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi,
Looking again at
travis-tool.shI feel like pieces of it could be simplified by usingdevtools::check(). I know that this will end up runningR CMD buildandR CMD checkin the end, but it deals with all the environment variables and options. After all,travis-tool.shusesdevtoolsin several parts already.Anyhow, in relation to #129, it took me a while to figure out why the outputs from
time R CMD checkandsystem.time(devtools::check())were different (not the final time, but the output). Turns out that I got used to the default behavior ofdevtools::check()which sets the environment variable_R_CHECK_TIMINGS_=10(see https://github.com/hadley/devtools/blob/master/R/check.r#L142 and http://cran.r-project.org/doc/manuals/r-release/R-ints.html#Tools).Cheers,
Leo
The text was updated successfully, but these errors were encountered: