search

NIDAP-Community / DSPWorkflow

https://nidap-community.github.io/DSPWorkflow/
MIT License
5 stars 3 forks source link

Most dependencies should be in `Imports` rather than `Depends` #127

Closed kelly-sovacool closed 1 year ago

kelly-sovacool commented 1 year ago

The package dependencies in the description file are listed under Depends, but most of them except the R version should likely go under Imports unless you have a good reason. See details here: https://r-pkgs.org/dependencies-mindset-background.html#sec-dependencies-imports-vs-depends

escauley commented 1 year ago

Sounds good! I will switch those over and test the package

escauley commented 1 year ago

All dependencies in the DESCRIPTIOn excpet for R have been moved to "imports" instead of "depends". The vignette ran successfully for this update and the changes were pushed to main