Roel Hogervorst

It is standard practice to disconnect from a database after use. I don't see an R function to do that. `DBI::disconnect()` for relational databases for instance could you add that...

If applied, this commit will create an extra remedy function: package linker. This allows you to select a piece of markdown and turn it into a link to the CRAN...

Upcoming changes in charlatan

1

Hi Matthew! I'm the new maintainer of the charlatan package that you use in one of your functions. I'm giving you a heads-up because I'm going to change a lot...

[optional] is it necessary to download all the scripts to ~/.dataretriever ?

2

I wonder if it is necessary to download all these jsons and python scripts to your ~/.dataretriever folder. This seems overkill/wasteful if you want to use 2 of the ~200...

Consider vignette of more examples

1

For novices it could be useful if you add some copy and paste examples of downloading data into different backends. That way people can follow the example and modify to...

encoding. of course it is encoding...

6

It would be very nice if the text parsing would default to utf-8, because I have something that doesn't seem to be right. 1001 nights Generous Dealing of Yahya Son...

Loading of subcomponents is slow. What if we lazy eval and delay until actually needed. https://stackoverflow.com/questions/35189760/using-delayed-assignment-in-a-function-how-do-i-send-the-promise-back-to-the-pa#35191961 https://rdocumentation.org/packages/base/versions/3.6.2/topics/delayedAssign With a delayed assignment, or promise ( base R)