nick-ulle/rstatic

Is 'rlang' a dependency?

clarkfitzg opened this issue · 1 comments

clarkfitzg commented

rstatic uses package rlang, but doesn't mention it in DESCRIPTION.

clarkf@poisson ~/dev/rstatic (master)
$ grep -r "rlang::" *
R/utilities.R:  dots = rlang::quos(...)
R/utilities.R:    if (rlang::quo_is_missing(elt))
R/utilities.R:      rlang::eval_tidy(elt)
nick-ulle commented

Yes. I was reluctant to add another dependency, but couldn't find any straightforward base R way to collect arguments to ... when some of them are missing.

I've added rlang to the imports and fixed a bug in the related code in 478f7af.