77
84%
master: 84%

Ran 20 Nov 2018 07:12AM UTC

Jobs 4

Files 3

Run time 2min

Badge

Embed ▾

Committed 20 Nov 2018 07:11AM UTC coverage: 84.3% (-0.05%) from 84.354%

Build # 77

Build Type

push

travis-ci

Committed by

ChrisHines

Commit Message

Drop invalid runes from keys instead of returning ErrInvalidKey

Experience has shown that including invalid runes (especially spaces) in
keys is the most common mistake when encoding. Returning an error for keys
containing invalid runes is not helpful because log packages often discard
the field or entire record when there is an encoding error. But missing
log data surprises application developers that don't understand why it
was discarded.

I considered two ways to handle bad runes in keys, optionally replacing
them with a different configurable rune, or dropping the bad runes from
they key. The first approach requires expanding the API and adding some
complexity to the implementation of Encoder. The second choice does not
change the API and, thanks to the strings.Map and bytes.Map functions,
slightly simplifies the implementation; so I chose it.

Run Details

494 of 586 relevant lines covered (84.3%)

106.45 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	77.1	20 Nov 2018 07:12AM UTC	84.3	Travis Job 77.1
2	77.2	20 Nov 2018 07:13AM UTC	84.3	Travis Job 77.2
3	77.3	20 Nov 2018 07:13AM UTC	84.3	Travis Job 77.3
4	77.4	20 Nov 2018 07:15AM UTC	84.3	Travis Job 77.4

Source Files on build 77

Coverage	∆	File	Lines	Relevant	Covered	Missed	Hits/Line

go-logfmt / logfmt / 77
84%
master: 84%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 77

go-logfmt / logfmt / 77 84% master: 84%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 77

go-logfmt / logfmt / 77
84%
master: 84%

README BADGES
x