Switch leaflet to depends

matthewlewis896 commented 3 years ago

remotes::install_github() fails if leaflet not installed.

> sessionInfo()
R version 4.0.3 (2020-10-10)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 10 x64 (build 19041)

Matrix products: default

locale:
[1] LC_COLLATE=English_United Kingdom.1252  LC_CTYPE=English_United Kingdom.1252   
[3] LC_MONETARY=English_United Kingdom.1252 LC_NUMERIC=C                           
[5] LC_TIME=English_United Kingdom.1252    

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base     

loaded via a namespace (and not attached):
 [1] rstudioapi_0.13   magrittr_2.0.1    usethis_2.0.0     devtools_2.3.2    pkgload_1.1.0     R6_2.5.0         
 [7] rlang_0.4.10      fansi_0.4.2       tools_4.0.3       pkgbuild_1.1.0    xfun_0.19         sessioninfo_1.1.1
[13] tinytex_0.27      cli_2.2.0         withr_2.4.0       ellipsis_0.3.1    remotes_2.2.0     assertthat_0.2.1 
[19] digest_0.6.27     rprojroot_2.0.2   lifecycle_0.2.0   crayon_1.3.4      processx_3.4.4    purrr_0.3.4      
[25] callr_3.5.1       fs_1.5.0          ps_1.4.0          testthat_3.0.0    memoise_1.1.0     glue_1.4.2       
[31] compiler_4.0.3    desc_1.2.0        prettyunits_1.1.1

Martin-Jung commented 3 years ago

Moved from suggests to depends

matthewlewis896 commented 3 years ago

Possibly should have been imports. Sorry I wrote this without thinking. Depends will work anyway. Can review later if needed

Martin-Jung commented 3 years ago

Aye, feel free to resolve issue. From my perspective the leaflet functionalities are rather optional...

Martin-Jung commented 3 years ago

Set to depends now.

iiasa / rN2000

Switch leaflet to depends #1