Refactoring to address #124 and #127 + clean up Imports

[elipousson]

I started working on this only to address #124 but, I wanted the package to be able to clear devtools::check(), so I ended up addressing a variety of other small issues at the same time. Most of the changes are focused on avoiding whole library imports + moving the @importFrom tags to the function or package documentation. Key changes to imports include:

• Adding cli, rlang, sp, utils to Imports and removing assertthat from Imports
• Adding geojsonio and stars to Suggests + moving leaflet.extras, leafpop, mapdeck, miniUI, and tmaptools from Imports to Suggests

I added cli and rlang to handle error and warning messages and use rlang::check_installed() for all the packages that are now in Suggests. I also switched the organization of the non-exported mapview functions to match the organization within the mapview repository.

Lastly, I addressed #124 by replacing the calls to dplyr::select_() with dplyr::select() and dplyr::all_of() and incorporated the fix that @warnes used to address #127 in his pending pull request. Hope this is welcome pull request but I'm happy to make changes if needed!

[elipousson]

@tim-salabim – no rush, of course, but just wanted to make sure you'd seen that this pull request existed. I also wanted to offer that I can always modify the pull request changes if there was some but not all of these changes that you'd like to see added to the package.

[tim-salabim]

Hi, sorry for the long silence. I'll get to this asap.

[elipousson]

No apologies needed! Just let me know if you have any questions or suggestions.

[dmorg78]

hey are there any updates? thanks for fixing this