epicontacts icon indicating copy to clipboard operation
epicontacts copied to clipboard

Is it necessary to have dplyr as a dependency?

Open zkamvar opened this issue 6 years ago • 2 comments

I did a search for dplyr in this repo and found this: https://github.com/reconhub/epicontacts/search?q=dplyr&type=Code

Basically, dplyr is only used for joins right now and I'm wondering if it may be better to just switch over to base::merge() to lighten up the dependencies.

zkamvar avatar Jul 27 '18 13:07 zkamvar

@zkamvar great point, i'd prefer to lose the dplyr dependency if possible ... i'm working on replacing the joins and printing with base functions

vpnagraj avatar Sep 07 '18 17:09 vpnagraj

FWIW, I think the printing is much better keeping the tibble format. I think it's worth keeping tibble around when we drop dplyr

zkamvar avatar Feb 14 '19 05:02 zkamvar