Closed olivroy closed 5 months ago
In favour of rlang since ellipsis is deprecated. https://rlang.r-lib.org/news/index.html#argument-intake-1-0-0
While running tests, I noticed some deprecation warnings, that I took care of
context(..)
expect_equivalent(...)
expect_equal(..., ignore_attr = TRUE)
aes_string("x", "y")
aes(.data$x, .data$y)
usethis::use_import_from("rlang", ".data")
covr
the test-coverage action can be fixed by following steps outlined in r-lib/actions#834
In favour of rlang since ellipsis is deprecated. https://rlang.r-lib.org/news/index.html#argument-intake-1-0-0
While running tests, I noticed some deprecation warnings, that I took care of
context(..)
in tests (the context is now derived from test file name)expect_equivalent(...)
->expect_equal(..., ignore_attr = TRUE)
aes_string("x", "y")
->aes(.data$x, .data$y)
(this required me to useusethis::use_import_from("rlang", ".data")
as suggested in ggplot2 in packages vignettecovr
from Suggests, as the test-coverage action installs the package already.