Skip to content

docs: document ConsensusParams#5165

Merged
mergify[bot] merged 2 commits intomasterfrom
anton/2836-consensus-params
Jul 29, 2020
Merged

docs: document ConsensusParams#5165
mergify[bot] merged 2 commits intomasterfrom
anton/2836-consensus-params

Conversation

@melekes
Copy link
Contributor

@melekes melekes commented Jul 29, 2020

Closes #2836

@melekes melekes requested a review from tessr as a code owner July 29, 2020 10:51
@auto-comment
Copy link

auto-comment bot commented Jul 29, 2020

👋 Thanks for creating a PR!

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

  • Wrote tests
  • Updated CHANGELOG_PENDING.md
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Updated relevant documentation (docs/) and code comments
  • Re-reviewed Files changed in the Github PR explorer
  • Applied Appropriate Labels

Thank you for your contribution to Tendermint! 🚀

@melekes melekes self-assigned this Jul 29, 2020
@melekes melekes added C:docs Component: Documentation S:automerge Automatically merge PR when requirements pass labels Jul 29, 2020
@codecov
Copy link

codecov bot commented Jul 29, 2020

Codecov Report

Merging #5165 into master will increase coverage by 3.79%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5165      +/-   ##
==========================================
+ Coverage   61.07%   64.87%   +3.79%     
==========================================
  Files         203      197       -6     
  Lines       20538    19260    -1278     
==========================================
- Hits        12544    12495      -49     
+ Misses       6920     5604    -1316     
- Partials     1074     1161      +87     
Impacted Files Coverage Δ
types/tx.go 89.65% <0.00%> (-3.45%) ⬇️
proxy/multi_app_conn.go 58.02% <0.00%> (ø)
cmd/tendermint/commands/root.go
abci/types/util.go
abci/client/client.go
behaviour/reporter.go
cmd/tendermint/commands/run_node.go
consensus/reactor.go
crypto/armor/armor.go
abci/example/kvstore/kvstore.go
... and 112 more

@@ -49,9 +49,28 @@ definition](https://github.com/tendermint/tendermint/blob/master/types/genesis.g
chain IDs, you will have a bad time. The ChainID must be less than 50 symbols.
- `consensus_params`
Copy link
Contributor

Choose a reason for hiding this comment

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

a minor note with a link to the consensus params in the spec?

@mergify mergify bot merged commit b5f0308 into master Jul 29, 2020
@mergify mergify bot deleted the anton/2836-consensus-params branch July 29, 2020 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C:docs Component: Documentation S:automerge Automatically merge PR when requirements pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

document ConsensusParams

2 participants