Don't double import rlang functions

[DavisVaughan]

Ideally I think we would not import all of rlang, and only import what we use. These throw mainly harmless warnings on R 3.1

rlang::eval_tidy

parsnip/NAMESPACE

Line 160 in 1f96393

importFrom(rlang,eval_tidy)

all of rlang
https://github.com/tidymodels/parsnip/blob/master/NAMESPACE#L126

location of importFrom

parsnip/R/arguments.R

Line 128 in fc366b6

#' @importFrom rlang eval_tidy

[topepo]

There are times that I do that so that people (including myself) can recognize where the function is coming from.

[DavisVaughan]

i think if we have multiple @importFrom calls floating around all over the place (for clarity), that would be fine and roxygen knows what to do with that. the minor problem is that we have both @import rlang and @importFrom rlang eval_tidy which both get added to the NAMESPACE. If we remove the @import rlang and just @importFrom the things we use, it should go away.

[github-actions]

This issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue.