remove lastresulthash from merklization in lastresult hash#9175
Merged
mergify[bot] merged 7 commits intomainfrom Aug 9, 2022
Merged
remove lastresulthash from merklization in lastresult hash#9175mergify[bot] merged 7 commits intomainfrom
mergify[bot] merged 7 commits intomainfrom
Conversation
Contributor
Author
|
ah its already in the spec but we were actually doing gas and gas wanted on top Line 225 in 3e40bfc |
cmwaters
approved these changes
Aug 8, 2022
Contributor
cmwaters
left a comment
There was a problem hiding this comment.
The spec repo is merged into this repo. You can find it under /spec. I have already made the updates
tychoish
approved these changes
Aug 8, 2022
samricotta
pushed a commit
that referenced
this pull request
Aug 9, 2022
remove gas from merklization in headers Im not sure where to change docs since main points to the spec repo but that repo is archived. Maybe someone can help me?
samricotta
pushed a commit
that referenced
this pull request
Aug 10, 2022
remove gas from merklization in headers Im not sure where to change docs since main points to the spec repo but that repo is archived. Maybe someone can help me?
samricotta
pushed a commit
that referenced
this pull request
Aug 11, 2022
remove gas from merklization in headers Im not sure where to change docs since main points to the spec repo but that repo is archived. Maybe someone can help me?
samricotta
pushed a commit
that referenced
this pull request
Aug 11, 2022
remove gas from merklization in headers Im not sure where to change docs since main points to the spec repo but that repo is archived. Maybe someone can help me?
samricotta
pushed a commit
that referenced
this pull request
Aug 12, 2022
remove gas from merklization in headers Im not sure where to change docs since main points to the spec repo but that repo is archived. Maybe someone can help me?
samricotta
pushed a commit
that referenced
this pull request
Aug 12, 2022
remove gas from merklization in headers Im not sure where to change docs since main points to the spec repo but that repo is archived. Maybe someone can help me?
samricotta
pushed a commit
that referenced
this pull request
Aug 16, 2022
remove gas from merklization in headers Im not sure where to change docs since main points to the spec repo but that repo is archived. Maybe someone can help me?
thanethomson
added a commit
that referenced
this pull request
Aug 24, 2022
…lastresult hash (#9313) * Revert "remove lastresulthash from merklization in lastresult hash (#9175)" This reverts commit bff63ae. Signed-off-by: Thane Thomson <connect@thanethomson.com> * Clarify wording in ABCI upgrade guidelines Signed-off-by: Thane Thomson <connect@thanethomson.com> Signed-off-by: Thane Thomson <connect@thanethomson.com>
38 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
remove gas from merklization in headers
Im not sure where to change docs since main points to the spec repo but that repo is archived. Maybe someone can help me?