Skip to content

Upkeep package #81

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 24 commits into from
Sep 17, 2024
Merged

Upkeep package #81

merged 24 commits into from
Sep 17, 2024

Conversation

nehill197
Copy link
Member

@nehill197 nehill197 commented Sep 17, 2024 •
edited
Loading

Perform upkeep on package #77 and replace requireNamespace() with rlang::check_installed() or rlang::is_installed() #72

nehill197 and others added 24 commits August 22, 2024 10:33
@nehill197
Replace requireNamespace() with rlang::check_installed()
756a984 
Closes #72
@nehill197
Update params documentation
882c791
@nehill197
upkeep
bb6905b
@nehill197
styler
d37109f
@nehill197
upkeep
22059db
@nehill197
Separate "translations" tests into their respective test files to mat…
2a4c6c5 
…ch .R files
@nehill197
Move ilog functions into their own R file to match test files
03e2dd8
@nehill197
Move ilog functions into their own R file to match test scripts
51c375a
@nehill197
Remove test environments from cran-comments.md (upkeep suggestion)
2a5a2d5
@nehill197
Update code coverage badge (no longer references master branch)
24acee4
@nehill197
Update lifecycle deprecation figures and docs
44b5c9b
@nehill197
Use pak instead of remotes in readme
fb27c87
@nehill197
Update logo sizes
de14535
@nehill197
Update documentation for ilog functions
7acee29
@nehill197
Add alternative text to vignette
1ca6e7e
@nehill197
Use tidy-style description format
9de2eed
@nehill197
Update code of conduct
d2d51e4
@nehill197
Update pkgdown changes from main branch
65a92ac
@nehill197
Merge branch 'main' into upkeep
59c0240
@dunkenwg
Removing .yaml from action titles
5a4ff55 
Hey Nicole, just fixing up the action names. Joe asked me to, and I thought this would be easier on the "upkeep" branch!
@dunkenwg
Removing .yaml from action titles
31fee2e
@dunkenwg
Removing .yaml from action titles
e080518
@nehill197
Remove duplicate VignetteBuilder line
bd40a97
@nehill197
Use rlang::is_installed() instead of requireNamespace for conditional…
227c7a6 
… examples and vignette code chunks
joethorley
joethorley approved these changes Sep 17, 2024
View reviewed changes
Copy link
Member

@joethorley joethorley left a comment

Choose a reason for hiding this comment

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

Looks great

@nehill197 nehill197 merged commit 5f2c336 into main Sep 17, 2024
10 checks passed
@nehill197 nehill197 deleted the upkeep branch September 17, 2024 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joethorley joethorley joethorley approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nehill197 @joethorley @dunkenwg