Skip to content

Release/v0.28.1#3153

Merged
ebuchman merged 2 commits intomasterfrom
release/v0.28.1
Jan 18, 2019
Merged

Release/v0.28.1#3153
ebuchman merged 2 commits intomasterfrom
release/v0.28.1

Conversation

@ebuchman
Copy link
Copy Markdown
Contributor

v0.28.1

* consensus: createProposalBlock function

* blockExecutor.CreateProposalBlock

- factored out of consensus pkg into a method on blockExec
- new private interfaces for mempool ("txNotifier") and evpool with one function each
- consensus tests still require more mempool methods

* failing test for CreateProposalBlock

* Fix bug in include evidece into block

* evidence: change maxBytes to maxSize

* MaxEvidencePerBlock

- changed to return both the max number and the max bytes
- preparation for #2590

* changelog

* fix linter

* Fix from review

Co-Authored-By: ebuchman <ethan@coinculture.info>
@ebuchman ebuchman requested a review from melekes as a code owner January 18, 2019 17:03
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #3153 into master will decrease coverage by 0.15%.
The diff coverage is 53.84%.

@@            Coverage Diff             @@
##           master    #3153      +/-   ##
==========================================
- Coverage   62.91%   62.75%   -0.16%     
==========================================
  Files         211      211              
  Lines       17438    17439       +1     
==========================================
- Hits        10971    10944      -27     
- Misses       5545     5567      +22     
- Partials      922      928       +6
Impacted Files Coverage Δ
consensus/replay_file.go 0% <0%> (ø) ⬆️
state/execution.go 70.63% <0%> (-2.17%) ⬇️
evidence/wire.go 66.66% <0%> (-33.34%) ⬇️
consensus/state.go 79.34% <100%> (-0.23%) ⬇️
evidence/pool.go 56.14% <100%> (ø) ⬆️
state/validation.go 65.81% <100%> (ø) ⬆️
evidence/store.go 90.36% <83.33%> (ø) ⬆️
privval/server.go 86.15% <0%> (-4.62%) ⬇️
privval/remote_signer.go 78.57% <0%> (-3.58%) ⬇️
consensus/reactor.go 66.23% <0%> (-1.19%) ⬇️
... and 3 more

@ebuchman ebuchman merged commit 0726329 into master Jan 18, 2019
iKapitonau pushed a commit to scrtlabs/tendermint that referenced this pull request Jul 16, 2024
…endermint#3176)

I updated all the old discord url to https://discord.gg/interchain or
https://discord.com/invite/interchain, since the vanity link for the
Cosmos discord has changed to interchain.

<hr>This is an automatic backport of pull request tendermint#3153 done by
[Mergify](https://mergify.com).

Co-authored-by: PolyMa <151764357+polymaer@users.noreply.github.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