search

grambank / rgrambank

R package to access and analyse Grambank's CLDF data
Apache License 2.0
3 stars 1 forks source link

add tidyverse and reshape2 to depdencies #11

Closed HedvigS closed 1 year ago

HedvigS commented 1 year ago

I think tidyverse and reshape2 should be installed and loaded for rgrambank.

SimonGreenhill commented 1 year ago

I'd rather add dependencies explicitly as needed (e.g. dplyr vs. all of tidyverse).

HedvigS commented 1 year ago

Okay. Once the current PRs are merged in, I can go check exactly which packages are used where.

HedvigS commented 1 year ago

so far I see that dplyr, readr, tidyr and reshape2 specifically would be good. Unless you want to re-write some things.

SimonGreenhill commented 1 year ago

I think most of these are now in the dependencies (i.e. tests won't work unless they're there)