Compact dockerfile

[VincentGuyader]

No description provided.

[Copilot]

Pull Request Overview

This PR adds two new options—single_file and set_golem.app.prod—to the Dockerfile helpers, updating code and documentation so that multi-stage Dockerfiles are generated by default and golem.app.prod=TRUE is set automatically.

• Introduce single_file and set_golem.app.prod parameters across all add_dockerfile_with_renv_* functions
• Implement branching logic for single‐ vs. two‐stage Dockerfile generation
• Update documentation and NEWS to describe the new defaults

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
man/dockerfiles.Rd	Added single_file and set_golem.app.prod to function docs
R/add_dockerfiles_renv.R	Added branching for multi-stage builds; new default options
NEWS.md	Noted new defaults in release notes
DESCRIPTION	Bumped dockerfiler dependency to ≥ 0.2.5

Comments suppressed due to low confidence (3)

R/add_dockerfiles_renv.R:449

• In add_dockerfile_with_renv_shinyproxy, the dockerfile_cmd does not include the golem.app.prod option, so set_golem.app.prod is ignored. Update the command to options(..., golem.app.prod = %s) to honor that flag.

"R -e \"options('shiny.port'=3838,shiny.host='0.0.0.0');library(%1$s);%1$s::run_app()\""

R/add_dockerfiles_renv.R:158

• [nitpick] This call uses get_golem_name(pkg = source_folder) but elsewhere you use get_golem_name(golem_wd = golem_wd). Standardize on one signature to avoid confusion and ensure consistency.

paste0(get_golem_name(pkg = source_folder), "_*.tar.gz")

man/dockerfiles.Rd:186

• [nitpick] Remove the extra space before the period and consider clarifying default notation (e.g., (default: TRUE)) to match other parameter descriptions.

If \code{TRUE} (by default), generate a single multi-stage Dockerfile .