rstudio / config

config package for R
https://rstudio.github.io/config/
256 stars 27 forks source link

Add warning on library(config) and make it explicit on git / docs #44

Closed CorneeldH closed 1 year ago

CorneeldH commented 1 year ago

Hi there, thanks for the package (and all the other stuff you guys do)! For a project we implemented renv and library calls etc and loaded packages at startup.

Then we ran into a R Studio error, apparently based on config. I found this issue: https://github.com/rstudio/config/issues/4

Two points:

Thanks for considering!

andrie commented 1 year ago

To throw a warning if the user calls library(config) is an interesting idea. I am investigating.

andrie commented 1 year ago

CRAN will most likely not accept a package that throws a warning on attaching the library. That said, throwing a message is possible and probably acceptable.

Will it be helpful in your use case to throw a message instead of a warning?

CorneeldH commented 1 year ago

Yeah, definitely.

I had in my code the config:: but added later on a library line to keep it simply in renv. A message would help.

Thanks for the quick response!