Skip to content

rdev 1.15.3#233

Merged
jabenninghoff merged 7 commits intomainfrom
update-todo
Nov 24, 2025
Merged

rdev 1.15.3#233
jabenninghoff merged 7 commits intomainfrom
update-todo

Conversation

@jabenninghoff
Copy link
Owner

  • Updated use_rdev_pkgdown() to disable the new pkgdown::build_llm_docs() by default (update use_rdev_pkgdown())

@jabenninghoff
Copy link
Owner Author

Re-run checks after GitHub rollback is complete, see actions/runner-images#13341 (comment)

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates rdev to version 1.15.3, which adds functionality to disable pkgdown's new build_llm_docs() feature by default when setting up pkgdown with rdev conventions.

Key Changes

  • Modified use_rdev_pkgdown() to set llm-docs: false in the pkgdown configuration
  • Added test coverage for the new llm-docs configuration
  • Updated documentation to reflect the new behavior
  • Updated package dependencies in renv.lock (R 4.5.2, pkgdown 2.2.0, and others)

Reviewed changes

Copilot reviewed 68 out of 73 changed files in this pull request and generated no comments.

Show a summary per file
File Description
R/setup.R Added pkg$'llm-docs' <- FALSE to disable LLM docs generation
tests/testthat/test-setup.R Added test assertion to verify llm-docs is disabled
tests/testthat/test-upkeep.R Updated expected upkeep checklist length for 2025
man/use_rdev_pkgdown.Rd Updated documentation to describe llm-docs disabling
renv.lock Updated R version and package dependencies
docs/* Generated documentation files from pkgdown build

The changes are appropriate and well-tested. The implementation correctly disables the llm-docs feature in pkgdown configuration, and the test coverage validates this behavior. The documentation accurately describes the new functionality.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jabenninghoff jabenninghoff merged commit c0130ed into main Nov 24, 2025
13 of 14 checks passed
@jabenninghoff jabenninghoff deleted the update-todo branch November 24, 2025 02:37
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.

2 participants