Skip to content

It's better read from genDoc than from state.validators when appHeight==0 in replay#2893

Merged
ebuchman merged 9 commits intotendermint:developfrom
james-ray:develop
Nov 26, 2018
Merged

It's better read from genDoc than from state.validators when appHeight==0 in replay#2893
ebuchman merged 9 commits intotendermint:developfrom
james-ray:develop

Conversation

@james-ray
Copy link
Contributor

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

fix #2892

@codecov-io
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (develop@72f86b5). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff             @@
##             develop    #2893   +/-   ##
==========================================
  Coverage           ?   62.18%           
==========================================
  Files              ?      212           
  Lines              ?    17309           
  Branches           ?        0           
==========================================
  Hits               ?    10764           
  Misses             ?     5637           
  Partials           ?      908
Impacted Files Coverage Δ
consensus/replay.go 57.14% <100%> (ø)

@james-ray james-ray changed the title It's better read from genDoc than state.validators when appHeight==0 It's better read from genDoc than state.validators when appHeight==0 in replay Nov 21, 2018
@james-ray james-ray changed the title It's better read from genDoc than state.validators when appHeight==0 in replay It's better read from genDoc than from state.validators when appHeight==0 in replay Nov 21, 2018
Copy link
Contributor

@melekes melekes left a comment

Choose a reason for hiding this comment

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

🍏

Seems logical to me.

Copy link
Contributor

@ebuchman ebuchman left a comment

Choose a reason for hiding this comment

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

Thanks for catching this!

@ebuchman ebuchman merged commit fe3b97f into tendermint:develop Nov 26, 2018
danil-lashin pushed a commit to danil-lashin/tendermint that referenced this pull request Nov 27, 2018
* develop:
  types: Emit tags from BeginBlock/EndBlock (tendermint#2747)
  Fix fast sync stack with wrong block tendermint#2457 (tendermint#2731)
  It's better read from genDoc than from state.validators when appHeight==0 in replay (tendermint#2893)
  update encoding spec (tendermint#2903)
  return back initially allowed level if we encounter allowed key (tendermint#2889)
  Handling integer IDs in JSON-RPC requests -- fixes tendermint#2366 (tendermint#2811)

# Conflicts:
#	blockchain/reactor_test.go
#	blockchain/store_test.go
#	node/node_test.go
#	types/events.go
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