-
-
Notifications
You must be signed in to change notification settings - Fork 48
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
use {pak} to parallel install with {renv} #343
Comments
Not sure I want to opt-in into an experimental feature, in particular due to
Not sure what the ETA is for that. In any case, it's good to know that this is going to be the default at some point. |
We could test it in a precommit branch. I thought that eventually the effort will be small as its only an option you might need to set. Not sure how easy it would to use a precommit branch though 🙂 |
This affects the installation procedure of the hooks, which is not handled by {precommit} (or any repositories that implements hooks), but in the upstream framework, i.e. here. So testing this would involve a change in the upstream, a release in the upstream at least, as we don't control the version of pre-commit that is ran on pre-commit.ci |
Locally, we could test it before by installing a development version of pre-commit from a branch (note pre-commit=upstream, {precommit} = R package). |
@pat-s offered to test with {mlr3viz}, which has a timeout because of dependency installation time > 120s. Potentially needs changes upstream in pre-commit.
The text was updated successfully, but these errors were encountered: