Skip to content

docs & spec: deduplicate block-structure.md#2331

Merged
ebuchman merged 1 commit intodevelopfrom
docs-dedup
Sep 5, 2018
Merged

docs & spec: deduplicate block-structure.md#2331
ebuchman merged 1 commit intodevelopfrom
docs-dedup

Conversation

@zramsay
Copy link
Contributor

@zramsay zramsay commented Sep 4, 2018

from #2143

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

@codecov-io
Copy link

Codecov Report

Merging #2331 into develop will increase coverage by 0.1%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           develop    #2331     +/-   ##
==========================================
+ Coverage    60.96%   61.07%   +0.1%     
==========================================
  Files          197      197             
  Lines        16282    16271     -11     
==========================================
+ Hits          9927     9938     +11     
+ Misses        5490     5471     -19     
+ Partials       865      862      -3
Impacted Files Coverage Δ
cmd/tendermint/commands/lite.go 11.53% <0%> (-1.62%) ⬇️
evidence/wire.go 100% <0%> (ø) ⬆️
p2p/peer.go 57.95% <0%> (ø) ⬆️
consensus/state.go 77.88% <0%> (+0.47%) ⬆️
consensus/reactor.go 74.09% <0%> (+1.07%) ⬆️

The `DataHash` can provide a nice check on the
[Data](https://godoc.org/github.com/tendermint/tendermint/types#Data)
returned in this same block. If you are subscribed to new blocks, via
tendermint RPC, in order to display or process the new transactions you
Copy link
Contributor

Choose a reason for hiding this comment

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

talk about the RPC doesn't belong in the spec. i'll fix in a separate PR

@ebuchman ebuchman merged commit 7f6bd5c into develop Sep 5, 2018
@ebuchman ebuchman deleted the docs-dedup branch September 5, 2018 16:21
iKapitonau pushed a commit to scrtlabs/tendermint that referenced this pull request Jul 10, 2024
tendermint#2338)

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


---


<details>
<summary>Mergify commands and options</summary>

<br />

More conditions and actions can be found in the
[documentation](https://docs.mergify.com/).

You can also trigger Mergify actions by commenting on this pull request:

- `@Mergifyio refresh` will re-evaluate the rules
- `@Mergifyio rebase` will rebase this PR on its base branch
- `@Mergifyio update` will merge the base branch into this PR
- `@Mergifyio backport <destination>` will backport this PR on
`<destination>` branch

Additionally, on Mergify [dashboard](https://dashboard.mergify.com) you
can:

- look at your merge queues
- generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com
</details>

Co-authored-by: Andy Nogueira <me@andynogueira.dev>
cboh4 pushed a commit to scrtlabs/tendermint that referenced this pull request Apr 7, 2025
tendermint#2339)

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


---


<details>
<summary>Mergify commands and options</summary>

<br />

More conditions and actions can be found in the
[documentation](https://docs.mergify.com/).

You can also trigger Mergify actions by commenting on this pull request:

- `@Mergifyio refresh` will re-evaluate the rules
- `@Mergifyio rebase` will rebase this PR on its base branch
- `@Mergifyio update` will merge the base branch into this PR
- `@Mergifyio backport <destination>` will backport this PR on
`<destination>` branch

Additionally, on Mergify [dashboard](https://dashboard.mergify.com) you
can:

- look at your merge queues
- generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com
</details>

Co-authored-by: Andy Nogueira <me@andynogueira.dev>
cboh4 pushed a commit to scrtlabs/tendermint that referenced this pull request Apr 7, 2025
tendermint#2340)

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


---


<details>
<summary>Mergify commands and options</summary>

<br />

More conditions and actions can be found in the
[documentation](https://docs.mergify.com/).

You can also trigger Mergify actions by commenting on this pull request:

- `@Mergifyio refresh` will re-evaluate the rules
- `@Mergifyio rebase` will rebase this PR on its base branch
- `@Mergifyio update` will merge the base branch into this PR
- `@Mergifyio backport <destination>` will backport this PR on
`<destination>` branch

Additionally, on Mergify [dashboard](https://dashboard.mergify.com) you
can:

- look at your merge queues
- generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com
</details>

Co-authored-by: Andy Nogueira <me@andynogueira.dev>
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