search

tiernanmartin / drakepkg

Demonstrate A Drake Workflow Package
Other
29 stars 5 forks source link

copy_pkg_files() shouldn't wipe out previous extdata/ #2

Closed tiernanmartin closed 6 years ago

tiernanmartin commented 6 years ago

... unless you want it to.

The default should probably be that plans/ and doc/ are overwritten every time but extdata/ only gets overwritten when the user tells the function to.


copy_pkg_files <- function(to = getwd(), 
                           overwrite_plans = TRUE, 
                           overwrite_doc = TRUE, 
                           overwrite_extdata = FALSE){
...
}
tiernanmartin commented 6 years ago

I changed my mind about overwrite_extdata = FALSE -- I think it's fine to default to TRUE because there shouldn't be any files in this directory anyway.