PIP-Technical-Team / pipdm

What the Package Does (One Line, Title Case)
https://pip-technical-team.github.io/pipdm/
Other
0 stars 0 forks source link

find_new_svy_data() is not context safe #70

Closed Aeilert closed 3 years ago

Aeilert commented 3 years ago

Make sure folder paths set in the ingestion pipeline are correctly parsed down / taken as inputs to the function.

Aeilert commented 3 years ago

Took another look at this. Parameter parsing is working.

Not sure if we should really use the is.null() logic for the cache directory, but closing this for now.

https://github.com/PIP-Technical-Team/pipdm/blob/a60d2948405bbcac0631bacb3b351ce7ba7bbe54/R/find_new_svy_data.R#L11-L30