Skip to content

Commit 27f58f8

Browse files
billdenneyclaude
andcommitted
fix: also require carrier for parallel builds
`purrr::in_parallel()` uses `carrier::crate()` to serialize worker closures and raises an undeclared-dependency error partway through if carrier isn't installed. Add carrier to Suggests and to the `rlang::check_installed()` pre-flight so the requirement is surfaced up-front with pkgdown's own error path. Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
1 parent ee37d03 commit 27f58f8

3 files changed

Lines changed: 3 additions & 2 deletions

File tree

DESCRIPTION

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@ Imports:
4646
xml2 (>= 1.3.1),
4747
yaml (>= 2.3.9)
4848
Suggests:
49+
carrier,
4950
covr,
5051
diffviewer,
5152
evaluate (>= 0.24.0),

R/build-articles.R

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,7 @@ build_articles <- function(
232232
quiet = quiet
233233
))
234234
} else {
235-
rlang::check_installed("mirai")
235+
rlang::check_installed(c("mirai", "carrier"))
236236
mirai::daemons(n_cores)
237237
withr::defer(mirai::daemons(0))
238238
unwrap_purrr_error(purrr::walk(

R/build-quarto-articles.R

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ build_quarto_articles <- function(
7272
output_dir = output_dir
7373
))
7474
} else {
75-
rlang::check_installed("mirai")
75+
rlang::check_installed(c("mirai", "carrier"))
7676
mirai::daemons(n_cores)
7777
withr::defer(mirai::daemons(0))
7878
unwrap_purrr_error(purrr::walk2(

0 commit comments

Comments
 (0)