Open
Conversation
Member
VincentGuyader
commented
Jun 21, 2025
- feat: In dock_from_renv, introduce build-arg RENV_PATHS_CACHE (default /root/.cache/R/renv) and propagate it as an ENV variable for configurable renv cache paths.
- feat: In dock_from_renv, update cache mounting to --mount=type=cache,id=renv-cache,target=${RENV_PATHS_CACHE} and use renv::restore(clean=FALSE) to speed up and harden the Docker build.
added 3 commits
June 22, 2025 00:27
…t /root/.cache/R/renv) and propagate it as an ENV variable for configurable renv cache paths.
feat: In dock_from_renv, update cache mounting to --mount=type=cache,id=renv-cache,target=${RENV_PATHS_CACHE} and use renv::restore(clean=FALSE) to speed up and harden the Docker build.
There was a problem hiding this comment.
Pull Request Overview
This PR introduces configurable renv cache paths by adding a build argument and environment variable and updates the cache mounting mechanism in the Docker build to speed it up and improve its robustness.
- Adds ARG and ENV declarations for RENV_PATHS_CACHE in Dockerfiles.
- Updates invocations of renv::restore to use clean = FALSE and dynamically set cache target.
- Updates version and documentation to reflect the new behavior.
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tests/testthat/renv_Dockerfile | Adds ARG/ENV for RENV_PATHS_CACHE and updates cache mount with dynamic target. |
| tests/testthat/Dockerfile | Removes an outdated Dockerfile in favor of the new configuration. |
| dockerfiler.Rproj | Adds a unique ProjectId to the project file. |
| R/dock_from_renv.R | Introduces ARG/ENV support and updates renv restore command for improved performance. |
| NEWS.md | Documents the new features and updates. |
| DESCRIPTION | Updates the version to reflect the new development release. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.