Skip to content

Clarify clean_doc build documentation#222

Closed
LeonidasZhak wants to merge 1 commit into
r-lib:mainfrom
LeonidasZhak:codex/clarify-clean-doc-docs
Closed

Clarify clean_doc build documentation#222
LeonidasZhak wants to merge 1 commit into
r-lib:mainfrom
LeonidasZhak:codex/clarify-clean-doc-docs

Conversation

@LeonidasZhak

Copy link
Copy Markdown

Fixes #187.

This clarifies the build(clean_doc = ) documentation for the currently implemented branches:

  • FALSE leaves inst/doc unchanged
  • NULL uses Config/build/clean-inst-doc when present
  • if still NULL, interactive sessions ask before removing files, while non-interactive sessions clean them

Checks:

  • Rscript -e 'tools::checkRd("man/build.Rd")'\n- git diff --check\n- R CMD build --no-build-vignettes .\n- R_LIBS_USER=/tmp/cli-r-lib _R_CHECK_FORCE_SUGGESTS_=false R CMD check --no-manual --ignore-vignettes --no-build-vignettes --no-tests pkgbuild_1.4.8.9000.tar.gz

@gaborcsardi

Copy link
Copy Markdown
Member

@LeonidasZhak Is this a purely LLM-generated pull request? Please stop opening LLM-generated pull requests on the r-lib and tidyverse repositories for now.

@LeonidasZhak

LeonidasZhak commented Jun 7, 2026

Copy link
Copy Markdown
Author

You are right. This was an LLM-assisted PR, and I should have been more careful before opening it. My intention was to explore whether LLM-assisted tools could help identify and fix small issues more efficiently, but I now understand that this created review burden and was not the right way to contribute here.
I’m closing this PR and will stop opening LLM-generated PRs on r-lib and tidyverse repositories for now. Sorry for adding noise to your review queue, and thank you for maintaining pkgbuild.

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

Successfully merging this pull request may close these issues.

clean_doc description in build help is incomplete

2 participants