Skip to content

Commit fab02f2

Browse files
docs: fold in Copilot's #115 review nits + restore lost VignetteIndexEntry (#116)
#115 was merged from commit 437c48a, before the 2dcdc73 force-push that fixed the vignette index entry, so that fix was lost; this restores it. Plus the three Copilot inline comments on #115: - `vignettes/dockerfiler.Rmd`: section heading "Create a Dockerfile from an renv.lock" -> "... from a renv.lock" (article). - `R/rthis.R` `@examples`, `vignettes/dockerfiler.Rmd` "Basic workflow" chunk, `README.Rmd`: the illustrative `install.packages(..., repos = "http://cran.irsn.fr/")` now uses the canonical HTTPS CRAN mirror `https://cloud.r-project.org` instead of a plaintext-HTTP regional mirror. - restored `%\VignetteIndexEntry{Getting started with dockerfiler}` so it matches the vignette's YAML `title:` (silences the rmarkdown title-mismatch warning). regenerated `man/r.Rd`; re-knit `README.md`. No code-logic change. `devtools::test()` on the touched test file: 0 failures. Vignette renders clean. No en/em-dashes in source.
1 parent 581d38a commit fab02f2

5 files changed

Lines changed: 7 additions & 7 deletions

File tree

R/rthis.R

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
#'
1313
#' @examples
1414
#' r(print("yeay"))
15-
#' r(install.packages("plumber", repos = "http://cran.irsn.fr/"))
15+
#' r(install.packages("plumber", repos = "https://cloud.r-project.org"))
1616
r <- function(code) {
1717
code <- paste(trimws(deparse(substitute(code))), collapse = " ")
1818
glue("R -e {shQuote(code, type = 'sh')}")

README.Rmd

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ my_dock$COMMENT("Install required R package.")
7373
Wrap your raw R Code inside the `r()` function to turn it into a bash command with `R -e`.
7474

7575
```{r}
76-
my_dock$RUN(r(install.packages("attempt", repos = "http://cran.irsn.fr/")))
76+
my_dock$RUN(r(install.packages("attempt", repos = "https://cloud.r-project.org")))
7777
```
7878

7979
Classical Docker stuffs:

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ Wrap your raw R Code inside the `r()` function to turn it into a bash
6464
command with `R -e`.
6565

6666
``` r
67-
my_dock$RUN(r(install.packages("attempt", repos = "http://cran.irsn.fr/")))
67+
my_dock$RUN(r(install.packages("attempt", repos = "https://cloud.r-project.org")))
6868
```
6969

7070
Classical Docker stuffs:

man/r.Rd

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vignettes/dockerfiler.Rmd

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ author: "Colin Fay"
44
date: "`r Sys.Date()`"
55
output: rmarkdown::html_vignette
66
vignette: >
7-
%\VignetteIndexEntry{getting_started}
7+
%\VignetteIndexEntry{Getting started with dockerfiler}
88
%\VignetteEngine{knitr::rmarkdown}
99
%\VignetteEncoding{UTF-8}
1010
---
@@ -53,7 +53,7 @@ my_dock$MAINTAINER("Colin FAY", "contact@colinfay.me")
5353
Wrap your raw R Code inside the `r()` function to turn it into a bash command with `R -e`.
5454

5555
```{r}
56-
my_dock$RUN(r(install.packages("attempt", repos = "http://cran.irsn.fr/")))
56+
my_dock$RUN(r(install.packages("attempt", repos = "https://cloud.r-project.org")))
5757
```
5858

5959
Classical Docker stuffs:
@@ -131,7 +131,7 @@ pulls Linux binaries from Posit Public Package Manager
131131
(`https://p3m.dev/cran/latest`). Pass `FROM = "rocker/r-base"` or
132132
`repos = c(CRAN = "https://cran.rstudio.com/")` to opt out.
133133

134-
## Create a Dockerfile from an renv.lock
134+
## Create a Dockerfile from a renv.lock
135135

136136
If your project uses `{renv}`, `dock_from_renv()` turns the `renv.lock`
137137
into a Dockerfile that restores the exact pinned versions.

0 commit comments

Comments
 (0)