ropensci / rentrez

talk with NCBI entrez using R
https://docs.ropensci.org/rentrez
Other
195 stars 38 forks source link

Travis errors when the vignette #71

Closed dwinter closed 8 years ago

dwinter commented 8 years ago

At the moment the Travis CI tests are failing. The error message suggests the vignette builder should be rmarkdown rather than knitr

Warning in engine$weave(file, quiet = quiet, encoding = enc) :
  The vignette engine knitr::rmarkdown is not available, because the rmarkdown package is not installed. Please install it.
Error: processing vignette 'rentrez_tutorial.Rmd' failed with diagnostics:
It seems you should call rmarkdown::render() instead of knitr::knit2html() because rentrez_tutorial.Rmd appears to be an R Markdown v2 document.
singmann commented 8 years ago

Instead of changing the VignetteBuilderone can also simply change to the container based infrastructure of Travis-CI by removing sudo. So instead of

sudo: required

Use:

sudo: false

(in my case it also works without either statement.)

The problem with changing to rmarkdown as VignetteBuilder is that then devtools::build_vignettes() does not work anymore.

singmann commented 8 years ago

Hello there, Actually, my previous response is wrong. It rather seems one needs to add rmarkdown to Suggests in the DESCRIPTION to get it to work with VignetteBuilder: knitr. Then it works with both the container based infrastructure and the "normal" one. Sorry for the confusion.