Skip to content
New issue

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

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

Already on GitHub? # to your account

Re-consider install_*() functions, in light of pak #1723

Closed
hadley opened this issue Jan 11, 2023 · 4 comments · Fixed by #2016
Closed

Re-consider install_*() functions, in light of pak #1723

hadley opened this issue Jan 11, 2023 · 4 comments · Fixed by #2016
Labels
readme 🏠 Readme, badges, ... tidy ✨ tools for the tidyverse team tidy-dev-day 🤓 Tidyverse Developer Day upkeep 🧹

Comments

@hadley
Copy link
Member

hadley commented Jan 11, 2023

Should the README templates switch to using pak? Are there other places we recommend devtools::install_github() that should switch to pak?

@hadley hadley added readme 🏠 Readme, badges, ... tidy ✨ tools for the tidyverse team upkeep labels Jan 17, 2023
@jennybc
Copy link
Member

jennybc commented Feb 1, 2023

Just consulted @gaborcsardi. For now, let's keep devtools::install_github() in the template. In terms of orienting more towards pak, we lean towards having install_github() switch from remotes to pak in the not-too-distant future.

@jennybc
Copy link
Member

jennybc commented Feb 1, 2023

Another idea from @gaborcsardi: perhaps devtools gains a more generic install_remote()-type function where the different sources are differentiated via pak's source DSL instead of via a bunch of different functions.

https://pak.r-lib.org/reference/pak_package_sources.html#shorthands

@hadley
Copy link
Member Author

hadley commented Feb 1, 2023

I still think we should consider at least switching to suggest pak for tidyverse packages, just as a way as exposing more people to it.

@jennybc jennybc changed the title Consider pak Re-consider install_*() functions, in light of pak Feb 1, 2023
@jennybc
Copy link
Member

jennybc commented Jul 23, 2024

@gaborcsardi Are you feeling more ready to put pak::pak() in usethis's README templates? I am.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
readme 🏠 Readme, badges, ... tidy ✨ tools for the tidyverse team tidy-dev-day 🤓 Tidyverse Developer Day upkeep 🧹
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants