Skip to content

Fix R CMD check and misc cleanups#445

Merged
hadley merged 7 commits into
tidyverse:mainfrom
shikokuchuo:fix-checks
Aug 20, 2025
Merged

Fix R CMD check and misc cleanups#445
hadley merged 7 commits into
tidyverse:mainfrom
shikokuchuo:fix-checks

Conversation

@shikokuchuo

@shikokuchuo shikokuchuo commented Aug 20, 2025

Copy link
Copy Markdown
Member

Closes #444. Closes #442.

  • Adds purrr and tidyr to suggests (required by the Zillow demo).
  • Updates DESCRIPTION for R >= 4.1 and Posit ROR.
  • Cleans up duplicate purrr functions as identified by @jonthegeek, and pulls the latest purrr standalone file.
  • Snapshot / spelling updates for a clean R CMD check.
  • Update GitHub actions to latest.

@shikokuchuo shikokuchuo left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@hadley this turned into a mini-upkeep to get this to check cleanly on CI. Tagging you as I can't seem to directly request a review.

jonthegeek added a commit to jonthegeek/rvest that referenced this pull request Aug 20, 2025
This won't pass checks until tidyverse#445 is merged, but this particular new test passes and should prevent regressions of this particular use case.
shikokuchuo pushed a commit to shikokuchuo/rvest that referenced this pull request Aug 20, 2025
This won't pass checks until tidyverse#445 is merged, but this particular new test passes and should prevent regressions of this particular use case.
@hadley hadley merged commit a3f4a31 into tidyverse:main Aug 20, 2025
13 checks passed
@hadley

hadley commented Aug 20, 2025

Copy link
Copy Markdown
Member

Thank you!

shikokuchuo pushed a commit to shikokuchuo/rvest that referenced this pull request Aug 20, 2025
This won't pass checks until tidyverse#445 is merged, but this particular new test passes and should prevent regressions of this particular use case.
shikokuchuo pushed a commit to shikokuchuo/rvest that referenced this pull request Aug 20, 2025
This won't pass checks until tidyverse#445 is merged, but this particular new test passes and should prevent regressions of this particular use case.
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.

Unstated dependencies in 'demo' map_chr and map_lgl defined twice

2 participants