Check if some required packages could be made suggested about covidregionaldata HOT 4 CLOSED

I think this rule of < 20 dependencies is counterproductive. I predicted people will just start depending on big meta-packages (like tidyverse) instead of smaller, more focused packages.

I suggest we do something like this (but not as hacky): many functions of the tidyverse are re-exported in many places and we can import them in a clever way to minimise dependencies. For example, tidyselect::any_of(), tidyselect::ends_with(), etc. are re-exported in dplyr so we can depend on dplyr only instead of tidyselect+dplyr.

I can submit a PR for this later this week.

from covidregionaldata.

This was changed in the development version of rlang: r-lib/rlang@aa587b0#diff-e4299ca7439da3d937d0a2b7b28e948378c040e1c337911055dbddf60af08e5c so it's probably just a mismatch between your local version and the version on GitHub Actions.

I don't think this was intended to be a re-export anyways so I removed it in d3c150b.

from covidregionaldata.

Hi Richard,

Firstly sorry for the lack of response here recently been very busy on another project and neglected everything else. Will get up to speed on the work you have been doing over the next few days and try and make some useful contributions.

For this issue, I agree limiting imports is a really good idea. My preferred solution would be yes to go with suggests and allow then a check for the package with a message about what is needed to be installed.

Auto-installing would make CRAN not happy I think and introducing code work arounnds might just make it harder to code things (though if possible could be useful).

from covidregionaldata.

If you can figure out why we're getting CMD check failures in reexports.Rd that would be welcome too. My local build (when I build the documentation) makes a pointer to \item{rlang}{\code{\link[rlang:dot-data]{.data}}} which fails; when I change it to \item{rlang}{\code{\link[rlang:tidyeval-data]{.data}}} it works, but I am now having to do this manually after building the documentation.

from covidregionaldata.