njtierney / geotargets

Targets extensions for geospatial data
http://geotargets.njtierney.com/
Other
61 stars 4 forks source link

`tar_terra_sprc()` simplification #65

Closed Aariq closed 7 months ago

Aariq commented 7 months ago

Removes some repeated or unnecessary code I noticed in tar_terra_sprc(). Implements suggestion in #61.

Aariq commented 7 months ago

I'm not sure semicolon_split() and semicolon_paste() were worth it because it seems difficult to use unexported functions inside read/write functions of custom formats.

njtierney commented 7 months ago

I feel you re the semicolon_paste/split functions - they feel like they should be useful but I can understand that in this context they don't really help!