Skip to content

docs: document canonical field#5166

Merged
mergify[bot] merged 3 commits intomasterfrom
anton/2769-canonical
Jul 30, 2020
Merged

docs: document canonical field#5166
mergify[bot] merged 3 commits intomasterfrom
anton/2769-canonical

Conversation

@melekes
Copy link
Contributor

@melekes melekes commented Jul 29, 2020

Closes #2769

Spec PR: tendermint/spec#125

@melekes melekes requested a review from tessr as a code owner July 29, 2020 11:14
@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
description: |
Commit results.

canonical switches from false to true for block H once block H+1 has been committed. Until then it's subjective and only reflects what this node has seen so far.
Copy link
Contributor

@tac0turtle tac0turtle Jul 29, 2020

Choose a reason for hiding this comment

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

link to spec? when merged

@mergify mergify bot merged commit 2b97e03 into master Jul 30, 2020
@mergify mergify bot deleted the anton/2769-canonical branch July 30, 2020 09:07
iKapitonau pushed a commit to scrtlabs/tendermint that referenced this pull request Oct 16, 2025
…ermint#5166)

---

#### PR checklist

- [x] Tests written/updated
- [x] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)
- [x] Updated relevant documentation (`docs/` or `spec/`) and code
comments
<hr>This is an automatic backport of pull request tendermint#5164 done by
[Mergify](https://mergify.com).

---------

Co-authored-by: Zachary Becker <zachary@interchainlabs.io>
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.

docs: canonical vs subjective commit

3 participants