1
78%
main: 77%

Ran 04 May 2018 11:54PM UTC

Files 26

Run time 1s

Badge

Embed ▾

Committed 04 May 2018 11:53PM UTC coverage: 76.236% (-0.2%) from 76.411%

Job # UPLOAD_COVERAGE=true

Build Type

push

travis-ci

Committed by

philpennock

Commit Message

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

Run Details

1126 of 1477 relevant lines covered (76.24%)

109.31 hits per line

PennockTech / tabular / 35 / 1
78%
main: 77%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 35.1 (UPLOAD_COVERAGE=true)

PennockTech / tabular / 35 / 1 78% main: 77%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 35.1 (UPLOAD_COVERAGE=true)

PennockTech / tabular / 35 / 1
78%
main: 77%

README BADGES
x