Skip to content

remove print statements#1950

Merged
xla merged 1 commit intodevelopfrom
1891-follow-up
Jul 11, 2018
Merged

remove print statements#1950
xla merged 1 commit intodevelopfrom
1891-follow-up

Conversation

@melekes
Copy link
Contributor

@melekes melekes commented Jul 11, 2018

Refs #1891 (comment)

  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Updated CHANGELOG.md

@codecov-io
Copy link

Codecov Report

Merging #1950 into develop will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #1950      +/-   ##
===========================================
- Coverage    60.78%   60.74%   -0.04%     
===========================================
  Files          198      198              
  Lines        15788    15788              
===========================================
- Hits          9597     9591       -6     
- Misses        5370     5373       +3     
- Partials       821      824       +3
Impacted Files Coverage Δ
consensus/state.go 76.62% <0%> (-0.74%) ⬇️
p2p/peer.go 60.81% <0%> (ø) ⬆️
consensus/reactor.go 78.68% <0%> (ø) ⬆️

Copy link
Contributor

@xla xla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 :octocat: :shipit: 🍡

@xla xla merged commit 4f7fac8 into develop Jul 11, 2018
@xla xla deleted the 1891-follow-up branch July 11, 2018 11:04
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.

3 participants