Skip to content

fix: commit the ndb in LoadVersionForOverwriting#640

Merged
tac0turtle merged 3 commits into
cosmos:masterfrom
yihuang:commit-rollback
Dec 6, 2022
Merged

fix: commit the ndb in LoadVersionForOverwriting#640
tac0turtle merged 3 commits into
cosmos:masterfrom
yihuang:commit-rollback

Conversation

@yihuang

@yihuang yihuang commented Dec 4, 2022

Copy link
Copy Markdown

need to backport to 0.19.x.
It was previously included in #636, separate out for easier review.

@yihuang yihuang requested a review from a team December 4, 2022 04:01
Comment thread CHANGELOG.md Outdated
@tac0turtle

Copy link
Copy Markdown
Contributor

was this a regression or a new change?

@yihuang

yihuang commented Dec 6, 2022

Copy link
Copy Markdown
Author

was this a regression or a new change?

it's a bug fix I believe.

Comment thread CHANGELOG.md Outdated
@tac0turtle

Copy link
Copy Markdown
Contributor

should we backport this to 0.19?

@yihuang

yihuang commented Dec 6, 2022

Copy link
Copy Markdown
Author

should we backport this to 0.19?

I think so

@tac0turtle tac0turtle merged commit e144f08 into cosmos:master Dec 6, 2022
@tac0turtle

Copy link
Copy Markdown
Contributor

amazing I will test on a chain before final release

@tac0turtle

Copy link
Copy Markdown
Contributor

@Mergifyio backport release/v0.19.x

mergify Bot pushed a commit that referenced this pull request Dec 7, 2022
(cherry picked from commit e144f08)

# Conflicts:
#	CHANGELOG.md
@mergify

mergify Bot commented Dec 7, 2022

Copy link
Copy Markdown
Contributor

backport release/v0.19.x

✅ Backports have been created

Details

@yihuang yihuang deleted the commit-rollback branch December 7, 2022 09:51
tac0turtle added a commit that referenced this pull request Dec 7, 2022
Co-authored-by: yihuang <huang@crypto.com>
Co-authored-by: Marko <marbar3778@yahoo.com>
ankurdotb pushed a commit to cheqd/iavl that referenced this pull request Feb 28, 2023
cosmos#643)

Co-authored-by: yihuang <huang@crypto.com>
Co-authored-by: Marko <marbar3778@yahoo.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants