PennockTech / tabular / 35

78%

main: 77%

push

travis-ci

JSON: handle structs which only have a String() method

JSON:
* Change cell JSONification:
  + Our API only requires a `String()` method on objects, not
    `MarshalText()` or `MarshalJSON()`; thus when an object in a cell is
    a struct, using `json.Marshal()` would result in `{}` if no exported
    fields, or something else when there are exported fields.
  + Calling `.String()` to get a fallback first also has the side-effect
    of updating any caching for derived values, so is a win.
  + Explicitly handle `{}` to signal falling back to the stringified
    form.  Do not handle other variants.
  + Updated docs a little; note that in a future API revision we might
    choose to prefer `MarshalText()` to `String()`
* Fix column identity error reporting

11 of 11 new or added lines in 1 file covered. (100.0%)

1126 of 1477 relevant lines covered (76.24%)

109.31 hits per line