-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
feat: add an option to choose between staged-dependencies and setup-r-dependencies #252
feat: add an option to choose between staged-dependencies and setup-r-dependencies #252
Conversation
Unit Tests Summary3 tests 3 ✅ 6s ⏱️ Results for commit bbaf1eb. ♻️ This comment has been updated with latest results. |
This all is awesome right now. I have one more comment that would help to speed up the things - is to use PPM instead of default CRAN. This would allow us to install from binaries as opposed to download sourced - build binary - install. The difference will be significant. UPDATE: |
I suggested a solution for using the binary packages from PPM in insightsengineering/setup-r-dependencies#6. |
This PR is still used for testing insightsengineering/setup-r-dependencies#8. Once it's merged and new uses: insightsengineering/setup-r-dependencies@setup-binary-package-repository-for-different-platform can be replaced with uses: insightsengineering/setup-r-dependencies@v1 and this PR can be merged. Edit: this is done. |
…ry-for-different-platforms" This reverts commit 67f8871.
…ore-description in rhub.yaml
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.
This looks good from my end 👍
Example of workflows adapted to select
setup-r-dependencies
as the dependency installation method: