Skip to content

Commit 020378a

Browse files
committed
Explicitly specify dplyr::across, rather than use (magic) across
While `dplyr` appears to automagically provide its own `across` inside `group_by`, we still want to explicitly use `dplyr::` to - satisfy package checks, - continue to work if `dplyr` removes this magic (e.g., if the maintainers don't like this magic ignoring any user-defined/attached non-`dplyr` `across` function), and - be clear to code readers&editors where the function comes from.
1 parent eaeb061 commit 020378a

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

R/get_test_data.R

+1-1
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ get_test_data <- function(recipe, x){
5151
.fns = ~ !is.na(.x)
5252
)
5353
) %>%
54-
epiprocess::group_by(across(dplyr::all_of(groups))) %>%
54+
epiprocess::group_by(dplyr::across(dplyr::all_of(groups))) %>%
5555
dplyr::slice_tail(n = max(max_lags) + 1) %>%
5656
epiprocess::ungroup()
5757

0 commit comments

Comments
 (0)