-
Notifications
You must be signed in to change notification settings - Fork 285
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.
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
Polish upkeep bullets #1905
Polish upkeep bullets #1905
Conversation
todo(" | ||
`usethis::use_package_doc()` | ||
Consider letting usethis manage your `@importFrom` directives here. | ||
`usethis::use_import_from()` is handy for this."), | ||
todo(" | ||
`usethis::use_testthat(3)` and upgrade to 3e, \\ | ||
[testthat 3e vignette](https://testthat.r-lib.org/articles/third-edition.html)"), | ||
todo(" | ||
Align the names of `R/` files and `test/` files for workflow happiness. | ||
The docs for `usethis::use_r()` include a helpful script. | ||
`usethis::rename_files()` may be be useful."), | ||
todo("`usethis::use_package_doc()`"), | ||
todo("`usethis::use_testthat(3)`"), | ||
todo("Align the names of `R/` files and `test/` files"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general, I like having detail, links, etc. But I can also imagine this is about making the items for the distant past more lean, yeah? And saving detail for the items we are more likely to be engaging with in the year of our lord 2023?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, exactly.
* [ ] Add alt-text to pictures, plots, etc; see https://posit.co/blog/knitr-fig-alt/ for examples | ||
|
||
### Eternal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea 💡
tests/testthat/_snaps/upkeep.md
Outdated
@@ -36,7 +36,7 @@ | |||
* [ ] Update email addresses *@rstudio.com -> *@posit.co | |||
* [ ] Update copyright holder in DESCRIPTION: `person("Posit Software, PBC", role = c("cph", "fnd"))` | |||
* [ ] `Run devtools::document()` to re-generate package-level help topic with DESCRIPTION changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* [ ] `Run devtools::document()` to re-generate package-level help topic with DESCRIPTION changes | |
* [ ] Run `devtools::document()` to re-generate package-level help topic with DESCRIPTION changes |
Not in the diff but something I just noticed.
No description provided.