-
Notifications
You must be signed in to change notification settings - Fork 217
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
Setup "wizard" #926
Comments
Yeah, IDK what the right place for this is. For renv people would need to add usethis to the renv project, which is not so great. If we are already using pak for installation, and pak will get some form of the dependency detection code as well, then maybe this should go in pak as well. |
Hmmm the renv + usethis issue is an example of more general problem, so we might need to solve that at a higher level. (No idea what that solution would look like yet). |
I wonder how this should look. Should it be a set of helper functions that add goodies to a repo, like typical usethis functions do, or some project template kind of thing, rather? |
I was thinking it would be a usethis function that generated the workflow yaml for you |
So maybe we could transfer this issue to usethis? |
Yeah, I think that's probably the right place for it to live. But this repo might still need to publish the components as individual yaml snippets. But let's wait on this a bit longer to see what @thomasp85's work looks like. |
I wonder if it would be useful to have some sort of setup wizard (which would maybe live in usethis) that allowed you to pick:
(Step one might go away eventually so depending on the timing we might not need to include it. ... Actually, even if we don't have a single action that can do any of the three, we can still detect locally so that the user doesn't need to select it. We can also filter down the options in 2 based on what file types are found.)
Then we'd slowly move the examples away from special case of R in production, reserving them for cases where all three pieces are roughly fixed (e.g. linting, documenting, checking).
The text was updated successfully, but these errors were encountered: