Closed cjyetman closed 8 months ago
For future reference, I think it would have been better to remove the "empty" keys entirely, especially for paths. One good way to do this while still specifying a complete spec would be to comment out those lines.
As it stands, the default behavior will cause errors and incorrect expectations. files will attempt to write to incorrect directories and the other parameters will cause unexpected behaviors in the resulting dataset. It would be better to have those explicitly error out when we attempt to use those keys as part of assignment to variables (possibly with a check on NULL)
relates to #160, as doing a check on expected_keys %in% names(config)
would be a good way to do a check for completeness while failing fast.
note: #145 will "officialize" the 2022Q4 values