Skip to content

Config option for JSON output formatter#2843

Merged
ebuchman merged 4 commits intodevelopfrom
StructuredLogging
Nov 15, 2018
Merged

Config option for JSON output formatter#2843
ebuchman merged 4 commits intodevelopfrom
StructuredLogging

Conversation

@melekes
Copy link
Contributor

@melekes melekes commented Nov 15, 2018

  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests (TESTED MANUALLY)
  • Updated CHANGELOG_PENDING.md

@codecov-io
Copy link

codecov-io commented Nov 15, 2018

Codecov Report

Merging #2843 into develop will decrease coverage by 0.08%.
The diff coverage is 45.45%.

@@             Coverage Diff             @@
##           develop    #2843      +/-   ##
===========================================
- Coverage    62.23%   62.14%   -0.09%     
===========================================
  Files          212      212              
  Lines        17255    17257       +2     
===========================================
- Hits         10738    10725      -13     
- Misses        5614     5623       +9     
- Partials       903      909       +6
Impacted Files Coverage Δ
config/toml.go 53.19% <ø> (ø) ⬆️
cmd/tendermint/commands/root.go 51.72% <0%> (-3.84%) ⬇️
config/config.go 68.34% <55.55%> (-0.27%) ⬇️
state/metrics.go 0% <0%> (-25%) ⬇️
consensus/reactor.go 65.93% <0%> (-1.74%) ⬇️
libs/clist/clist.go 66.66% <0%> (-1.52%) ⬇️
blockchain/pool.go 65.73% <0%> (-0.7%) ⬇️
consensus/wal_generator.go 79.43% <0%> (+0.27%) ⬆️
node/node.go 64.18% <0%> (+0.91%) ⬆️
... and 1 more

@ebuchman ebuchman merged commit 06225e3 into develop Nov 15, 2018
@ebuchman ebuchman deleted the StructuredLogging branch November 15, 2018 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants