Skip to content

rdev 1.15.4#234

Merged
jabenninghoff merged 5 commits intomainfrom
update-lintr
Dec 4, 2025
Merged

rdev 1.15.4#234
jabenninghoff merged 5 commits intomainfrom
update-lintr

Conversation

@jabenninghoff
Copy link
Owner

  • Updated for lintr 3.3.0

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.4 to support lintr 3.3.0. The changes include package dependency updates in renv.lock, code modifications to comply with new lintr rules, and documentation updates to reflect lintr 3.3.0 API changes.

Key Changes:

  • Updated lintr from 3.2.0 to 3.3.0-1 with new dependencies and linters
  • Modified R code to remove explicit return() statements per lintr's return_linter
  • Added linter suppression comment where explicit return is kept for clarity
  • Updated documentation for lint_all() to reflect lintr 3.3.0 parameter changes

Reviewed changes

Copilot reviewed 65 out of 67 changed files in this pull request and generated no comments.

Show a summary per file
File Description
renv.lock Updated package versions: htmltools (0.5.8.1→0.5.9), lintr (3.2.0→3.3.0-1), shiny (1.11.1→1.12.0), testthat (3.3.0→3.3.1), xml2 (1.5.0→1.5.1), yaml (2.3.10→2.3.11)
R/utils.R Added nolint directive to suppress return_linter where explicit return aids clarity
R/to_document.R Removed explicit return() statement per lintr conventions
R/setup.R Removed explicit return() statement per lintr conventions
R/build.R Removed explicit return() statement per lintr conventions
man/lint_all.Rd Updated parameter documentation to match lintr 3.3.0 changes (parse_settings, show_progress)
docs/ Auto-generated documentation updates reflecting version bump and documentation changes
DESCRIPTION Version bump 1.15.3 → 1.15.4
NEWS.md Added changelog entry for lintr 3.3.0 update

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

@jabenninghoff jabenninghoff merged commit 4ab880a into main Dec 4, 2025
13 checks passed
@jabenninghoff jabenninghoff deleted the update-lintr branch December 4, 2025 20:14
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