Skip to content

fix: Suggest lattice so that it's available for tests - #1231

Merged
gadenbuie merged 5 commits into
mainfrom
tests/fix-needs-lattice
Sep 22, 2025
Merged

fix: Suggest lattice so that it's available for tests#1231
gadenbuie merged 5 commits into
mainfrom
tests/fix-needs-lattice

Conversation

@gadenbuie

Copy link
Copy Markdown
Member

Adds in {lattice} so that it's available for the rmd-skeleton test.

@gadenbuie
gadenbuie merged commit 220fa5d into main Sep 22, 2025
1 check passed
@gadenbuie
gadenbuie deleted the tests/fix-needs-lattice branch September 22, 2025 15:19
@cpsievert

Copy link
Copy Markdown
Collaborator

More so for future reference, but I tend to prefer Config/Needs/check: to Suggests: in these situations because of CRAN's limit on the number of suggested dependencies (IIRC it's ~20?)

@gadenbuie

Copy link
Copy Markdown
Member Author

Yeah, I looked for that first but we don't have that set up here. Also, I thought the limits were on Imports and not Suggests packages

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