Noticed while working on something else. I think these are all (?) places where devtools could use cli's {.run ...} construct instead of just .code. Some (but not all) of these packages are the blessed ones, but it's neutral-to-positive either way. All the sitrep.R and install.R stuff is involved in a PR I'm about to make, but I want to deal with the others when I'm done.
| File |
Line |
Current |
Suggested |
| R/sitrep.R |
241 |
{.code {update_code}} |
{.run {update_code}} |
| R/sitrep.R |
251 |
{.code {update_code}} |
{.run {update_code}} |
| R/install.R |
257 |
{.code pak::local_install_dev_deps()} |
{.run pak::local_install_dev_deps()} |
| R/test.R |
35 |
{.code usethis::use_testthat()} |
{.run usethis::use_testthat()} |
| R/release.R |
222 |
{.code use_cran_comments()} |
{.run use_cran_comments()} |
FYI here are the current blessed packages:
- testthat
- rlang
- devtools
- usethis
- pkgload
- pkgdown
Blessed packages in RStudio: https://github.com/rstudio/rstudio/blob/53842c110064f12923336c702d2ecd186f6c452c/src/cpp/session/modules/SessionPackages.R#L1166-L1175
Blessed packages in Positron: https://github.com/posit-dev/positron/blob/6fe4953badda8756c4e2abd1096ad40917d8facd/extensions/positron-r/src/hyperlink.ts#L89-L92
Noticed while working on something else. I think these are all (?) places where devtools could use cli's
{.run ...}construct instead of just.code. Some (but not all) of these packages are the blessed ones, but it's neutral-to-positive either way. All thesitrep.Randinstall.Rstuff is involved in a PR I'm about to make, but I want to deal with the others when I'm done.FYI here are the current blessed packages:
Blessed packages in RStudio: https://github.com/rstudio/rstudio/blob/53842c110064f12923336c702d2ecd186f6c452c/src/cpp/session/modules/SessionPackages.R#L1166-L1175
Blessed packages in Positron: https://github.com/posit-dev/positron/blob/6fe4953badda8756c4e2abd1096ad40917d8facd/extensions/positron-r/src/hyperlink.ts#L89-L92