`tar_format()` functions: Use `substitute()` into function body rather than `body<-` #43

brownag · 2024-03-19T02:50:58Z

Uses a simpler approach to updating the read/write function templates used in format creation methods, thanks to Will Landau for suggestion
Fixes Apparent bugs / problems with function body modification with covr+testthat+tar_test() #38 by avoiding the problematic syntax, normal tests and tests in covr now work

njtierney · 2024-03-19T03:00:09Z

Ah, fantastic!

- substitute in options, read/write functions following njtierney#43 - add support for reading netCDF via `stars::read_ncdf()` using njtierney#43 - consolidate option getter usage for njtierney#34

Use substitute() into function body rather than body<-; fixes njt…

15e798c

…ierney#38

njtierney merged commit 8191107 into njtierney:master Mar 19, 2024
6 checks passed

njtierney mentioned this pull request Mar 19, 2024

code coverage is at 0%? #31

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

`tar_format()` functions: Use `substitute()` into function body rather than `body<-` #43

`tar_format()` functions: Use `substitute()` into function body rather than `body<-` #43

brownag commented Mar 19, 2024

njtierney commented Mar 19, 2024

tar_format() functions: Use substitute() into function body rather than body<- #43

tar_format() functions: Use substitute() into function body rather than body<- #43

Conversation

brownag commented Mar 19, 2024

njtierney commented Mar 19, 2024

`tar_format()` functions: Use `substitute()` into function body rather than `body<-` #43

`tar_format()` functions: Use `substitute()` into function body rather than `body<-` #43