Skip to content

spec: Add additional information about validator ordering (total power) #2715

Merged
andynog merged 3 commits intomainfrom
andy/779-spec-vote-info
Apr 4, 2024
Merged

spec: Add additional information about validator ordering (total power) #2715
andynog merged 3 commits intomainfrom
andy/779-spec-vote-info

Conversation

@andynog
Copy link
Collaborator

@andynog andynog commented Apr 3, 2024

close: #779

This PR adds additional information on ExtendedCommitInfo and CommitInfo data types about the validator set ordering (total power) guarantees.

PR checklist

  • Tests written/updated
  • Changelog entry added in .changelog (we use unclog to manage our changelog)
  • Updated relevant documentation (docs/ or spec/) and code comments
  • Title follows the Conventional Commits spec

@andynog andynog self-assigned this Apr 3, 2024
@andynog andynog requested review from a team as code owners April 3, 2024 20:51
@andynog andynog requested a review from adizere April 3, 2024 20:51
Copy link
Contributor

@adizere adizere left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the swift turnover Andy!

Should we backport this into 0.38 and v1 branches ?

@andynog
Copy link
Collaborator Author

andynog commented Apr 4, 2024

Thanks @adizere, I think we should backport at least to v1. For v0.38.x since it's a spec and we might have many things in v1 that is not part of v0.38 that I feel it would be better just to do a separate PR for v0.38 (which would be faster too).

@andynog andynog added this pull request to the merge queue Apr 4, 2024
Merged via the queue into main with commit 0349b99 Apr 4, 2024
@andynog andynog deleted the andy/779-spec-vote-info branch April 4, 2024 18:52
mergify bot pushed a commit that referenced this pull request Apr 4, 2024
…r) (#2715)

close: #779

This PR adds additional information on `ExtendedCommitInfo` and
`CommitInfo` data types about the validator set ordering (total power)
guarantees.

#### PR checklist

- [ ] Tests written/updated
- [ ] 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
- [X] Title follows the [Conventional
Commits](https://www.conventionalcommits.org/en/v1.0.0/) spec

(cherry picked from commit 0349b99)
andynog added a commit that referenced this pull request Apr 4, 2024
…r) (backport #2715) (#2720)

close: #779 

This PR adds additional information on `ExtendedCommitInfo` and
`CommitInfo` data types about the validator set ordering (total power)
guarantees.

#### PR checklist

- [ ] Tests written/updated
- [ ] 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
- [X] Title follows the [Conventional
Commits](https://www.conventionalcommits.org/en/v1.0.0/) spec
<hr>This is an automatic backport of pull request #2715 done by
[Mergify](https://mergify.com).

Co-authored-by: Andy Nogueira <me@andynogueira.dev>
@andynog andynog added the backport-to-v0.38.x Tell Mergify to backport the PR to v0.38.x label Apr 5, 2024
mergify bot pushed a commit that referenced this pull request Apr 5, 2024
…r) (#2715)

close: #779

This PR adds additional information on `ExtendedCommitInfo` and
`CommitInfo` data types about the validator set ordering (total power)
guarantees.

#### PR checklist

- [ ] Tests written/updated
- [ ] 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
- [X] Title follows the [Conventional
Commits](https://www.conventionalcommits.org/en/v1.0.0/) spec

(cherry picked from commit 0349b99)
andynog added a commit that referenced this pull request Apr 5, 2024
…r) (backport #2715) (#2724)

close: #779 

This PR adds additional information on `ExtendedCommitInfo` and
`CommitInfo` data types about the validator set ordering (total power)
guarantees.

#### PR checklist

- [ ] Tests written/updated
- [ ] 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
- [X] Title follows the [Conventional
Commits](https://www.conventionalcommits.org/en/v1.0.0/) spec
<hr>This is an automatic backport of pull request #2715 done by
[Mergify](https://mergify.com).

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

backport-to-v0.38.x Tell Mergify to backport the PR to v0.38.x

Projects

None yet

Development

Successfully merging this pull request may close these issues.

spec: Clearly document the ordering within VoteInfo and ExtendedVoteInfo

2 participants