Skip to content

Jae/lower case round state#1483

Merged
melekes merged 2 commits intodevelopfrom
jae/lower_case_round_state
Apr 24, 2018
Merged

Jae/lower case round state#1483
melekes merged 2 commits intodevelopfrom
jae/lower_case_round_state

Conversation

@jaekwon
Copy link
Contributor

@jaekwon jaekwon commented Apr 21, 2018

Makes it consistent w/ everything else. This inconsistency was encountered during dump_consensus_state visualization in tendermint/tendermint-vue

@jaekwon jaekwon changed the base branch from master to develop April 21, 2018 06:22
@codecov-io
Copy link

codecov-io commented Apr 21, 2018

Codecov Report

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

@@             Coverage Diff             @@
##           develop    #1483      +/-   ##
===========================================
- Coverage    63.83%   63.72%   -0.12%     
===========================================
  Files          102      102              
  Lines         9460     9460              
===========================================
- Hits          6039     6028      -11     
- Misses        2862     2869       +7     
- Partials       559      563       +4
Impacted Files Coverage Δ
consensus/reactor.go 72.59% <0%> (-1.32%) ⬇️
consensus/state.go 77.34% <0%> (-0.41%) ⬇️
p2p/peer.go 63.76% <0%> (ø) ⬆️
blockchain/pool.go 68.75% <0%> (+0.69%) ⬆️

@melekes melekes merged commit 5361073 into develop Apr 24, 2018
@melekes melekes deleted the jae/lower_case_round_state branch April 24, 2018 07:57
firelizzard18 pushed a commit to AccumulateNetwork/tendermint that referenced this pull request Feb 1, 2024
Signed-off-by: Thane Thomson <connect@thanethomson.com>
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