geomarker-io / codec

Community Data Explorer for Cincinnati
https://geomarker.io/codec/
GNU General Public License v3.0
4 stars 3 forks source link

check metadata #22

Closed erikarasnick closed 1 year ago

erikarasnick commented 1 year ago

I don't think {assertr} is a good fit here, since was designed for tibbles and most elements of the metadata are single values stored in a list. {assertr} might work for checking the data structure or even the schema portion of the metadata.

cole-brokamp commented 1 year ago

so, circling back to our conversation today, you will merge these check functions into main after moving them into R/codec.R, right?

erikarasnick commented 1 year ago

@cole-brokamp Should I add the rest of the check functions we talked about yesterday under this PR? or merge what we have so far?

cole-brokamp commented 1 year ago

I think just move what you have here into that file and merge it that way.

cole-brokamp commented 1 year ago

I think just move what you have here into that file and merge it that way.

scratch that! I misunderstood. Yes, let's just merge this in and continue to work on the other check functions directly on main.