Switch to `tidyselect::eval_select()`

tidyverse / vroom

Fast reading of delimited files

https://vroom.r-lib.org

Other

621 stars 60 forks source link

Switch to `tidyselect::eval_select()` #459

Open jennybc opened 2 years ago

jennybc commented 2 years ago

vroom currently uses tidyselect::vars_select(), but it's marked as deprecated / questioning. Instead we should use tidyselect::eval_select().

https://tidyselect.r-lib.org/reference/vars_select.html

jennybc commented 2 years ago

Upon further scrutiny, it feels like the internal handling of col_select needs a general refresh / simplification. For example, vroom_enquo() was last touched 3 years ago and tidy eval + the tidyselect package has matured a lot since then.