revdepcheck icon indicating copy to clipboard operation
revdepcheck copied to clipboard

Re-enable checking of vignettes?

Open HenrikBengtsson opened this issue 5 years ago • 2 comments

Hi, I just ran into a case where CRAN's incoming reverse-dependency checks caught a bug in my submitted package that revealed itself only in a vignette in one of the 138 reverse-depency packages.

After some troubleshooting, I figured out that revdepcheck:revdep_check() skips the vignettes for its checks;

https://github.com/r-lib/revdepcheck/blob/61ab33e6adc3cfcb5d51012b8ad7e94fd99a092c/R/check.R#L32

I don't see an obvious way to re-enable vignette checks. Could you please add this as an argument that can be overridden, e.g. args = c("--no-manual", "--no-build-vignettes")?

HenrikBengtsson avatar Sep 21 '20 19:09 HenrikBengtsson

Bump! I just got bitten by this ...

bbolker avatar Mar 13 '23 19:03 bbolker

For the record, --no-build-vignettes still runs all the code in the vignettes, only does not build the output PDF or HTML. Nevertheless, I don't mind if we make this configurable with an env var.

gaborcsardi avatar Mar 13 '23 19:03 gaborcsardi