Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expose skip of inst/doc overwrite check directly to rcmdcheck() #224

Open
MichaelChirico opened this issue Aug 18, 2024 · 0 comments
Open

Comments

@MichaelChirico
Copy link
Contributor

MichaelChirico commented Aug 18, 2024

Follow-up to #130 / #155.

AFAICT that solution is to add a new entry to the package DESCRIPTION telling it "it's OK to overwrite inst/doc".

However, I am working on a use case where I clone the package from the GitHub mirror (e.g. github.com/cran/xts) and then check it. The GitHub mirror contains the unpacked tar.gz version of the packages, which always has the inst/doc folder (if there are vignettes). Since I'm just running R CMD check on a git clone & then deleting the package, I definitely don't care if inst/doc gets deleted.

Essentially, I am not the package maintainer, so editing the package DESCRIPTION is a very obscure way to go about this.

For now, I will just unlink('inst/doc', recursive=TRUE) to my clone, but IMHO it would be a better design to expose some way to say "yes" to this prompt directly in rcmdcheck().

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant