Have vignettes not be deleted if "BuildVignettes: no" in DESCRIPTION

[schloerke]

While the 95% case is to build vignettes every time from source, I'd like to be able to not have to manually set vignettes = FALSE in pkgbuild::build if in the DESCRIPTION file states BuildVignettes: no.

This would be very handy.

[schloerke]

@alandipert for shinyloadtest

[jimhester]

Ok, did you try it? pkgbuild is just delegating to R CMD build, so this should already work and does seem to in a simple test package.

[schloerke]

Yes. I'm trying on repo rstudio/sortable@pkgdown

If I call devtools::check(), it still asks if it can delete ./inst/doc, which I don't want it to do.
If I respond, "No", the build stops.

Calling devtools::check(vignettes = FALSE) gives me the desired behavior of not asking about deleting ./inst/doc and not building the vignettes.

[jimhester]

deleting inst/doc is different than building the vignettes...

[gaborcsardi]

I think it would be great to document the newly suggested vignette workflow, and also how people show go about not rebuilding the vignettes all the time.

[jimhester]

You can now use clean_doc = FALSE to avoid cleaning the inst/doc folder.