Skip to content

Commit 01d77aa

Browse files
authored
Merge pull request #468 from cmu-delphi/ndefries/dots-warning
Assign to dots using `assign` to avoid CI warning
2 parents 7fd2119 + 2709580 commit 01d77aa

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

R/grouped_epi_archive.R

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -317,7 +317,10 @@ epix_slide.grouped_epi_archive <- function(x, f, ..., before, ref_time_values,
317317

318318
f <- quos[[1]]
319319
new_col <- sym(names(rlang::quos_auto_name(quos)))
320-
... <- missing_arg() # nolint: object_usage_linter. magic value that passes zero args as dots in calls below
320+
# Magic value that passes zero args as dots in calls below. Equivalent to
321+
# `... <- missing_arg()`, but use `assign` to avoid warning about
322+
# improper use of dots.
323+
assign("...", missing_arg())
321324
}
322325

323326
f <- as_slide_computation(f, ...)

R/slide.R

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -304,7 +304,10 @@ epi_slide <- function(x, f, ..., before, after, ref_time_values,
304304

305305
f <- quos[[1]]
306306
new_col <- sym(names(rlang::quos_auto_name(quos)))
307-
... <- missing_arg() # magic value that passes zero args as dots in calls below # nolint: object_usage_linter
307+
# Magic value that passes zero args as dots in calls below. Equivalent to
308+
# `... <- missing_arg()`, but use `assign` to avoid warning about
309+
# improper use of dots.
310+
assign("...", missing_arg())
308311
}
309312

310313
f <- as_slide_computation(f, ...)

0 commit comments

Comments
 (0)