importBC_data() - suggestion: load 'plyr' before 'dplyr' about envair

[jeromerobles]

New version (envair 0.1.0.0), now removes the loading of libraries

[ateucher]

Hope you don't mind me chiming in here! It's not considered good practice to load libraries in a package (in fact if you tried to upload to CRAN they won't allow it). The reason is that it is altering the user's environment in ways that they have not asked for, and is considered "user-hostile" (perhaps a bit harsh, but @GailR's experience proves the point).

Instead you should list all of the packages that envair uses in the Imports: field in the DESCRIPTION file. That ensures that all of the packages will be installed and available for your package to use. It looks like you normally specify the functions using the package::function() syntax, which is great - that is how you call a function without loading the package using library() or require(). So by listing the dependencies in the DESCRIPTION you will ensure the required packages are installed, and then using the package::function() you can call all the functions you need without loading the libraries.

Some useful links:
https://r-pkgs.org/description.html#dependencies
https://r-pkgs.org/namespace.html

Hope this is helpful!

[aseagram]

+1 upvote to have this issue fixed ASAP, as described above. @jeromerobles