is it actually necessary to have a config file for this? I ask this since nearly all the configuration settings are hardcoded into the targets file from lines 35-86. I know that part of the reason for this is to use retrieve_target() as written originally, but since you're giving the rds name and id explicitly, that function could be simplified to just drive_download() and then readRDS() given that the file name and driveid is already stated earlier. There are a couple of ways you could probably reduce LOC and reduce the need of that config file, but that's truly just if you wanted to do so.
Originally posted by @steeleb in #2 (comment)
is it actually necessary to have a config file for this? I ask this since nearly all the configuration settings are hardcoded into the targets file from lines 35-86. I know that part of the reason for this is to use
retrieve_target()as written originally, but since you're giving the rds name and id explicitly, that function could be simplified to justdrive_download()and thenreadRDS()given that the file name and driveid is already stated earlier. There are a couple of ways you could probably reduce LOC and reduce the need of that config file, but that's truly just if you wanted to do so.Originally posted by @steeleb in #2 (comment)