Bump TX version v7#79
Conversation
07cf380 to
8d474f0
Compare
masihyeganeh
left a comment
There was a problem hiding this comment.
@masihyeganeh reviewed 327 files and all commit messages, and made 1 comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @metalarm10, @miladz68, @TxCorpi0x, and @ysv).
build/tools/tools.go line 25 at r1 (raw file):
Hermes tools.Name = "hermes" // TXdV600 is the version of txd used for testing chain upgrades. // TODO (v8): Should be renamed to TXdV700.
nit: Don't need this TODO anymore
TxCorpi0x
left a comment
There was a problem hiding this comment.
@TxCorpi0x made 1 comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @masihyeganeh, @metalarm10, @miladz68, and @ysv).
build/tools/tools.go line 25 at r1 (raw file):
Previously, masihyeganeh (Masih Yeganeh) wrote…
nit: Don't need this TODO anymore
This is for the next version, it is todo v8
masihyeganeh
left a comment
There was a problem hiding this comment.
@masihyeganeh made 1 comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @metalarm10, @miladz68, @TxCorpi0x, and @ysv).
build/tools/tools.go line 25 at r1 (raw file):
Previously, TxCorpi0x wrote…
This is for the next version, it is todo v8
I added this to remember to change it from Coreum to TX. We don't need it anymore
TxCorpi0x
left a comment
There was a problem hiding this comment.
@TxCorpi0x made 1 comment and resolved 1 discussion.
Reviewable status: 326 of 327 files reviewed, all discussions resolved (waiting on @masihyeganeh, @metalarm10, @miladz68, and @ysv).
build/tools/tools.go line 25 at r1 (raw file):
Previously, masihyeganeh (Masih Yeganeh) wrote…
I added this to remember to change it from Coreum to TX. We don't need it anymore
Done
masihyeganeh
left a comment
There was a problem hiding this comment.
@masihyeganeh reviewed 1 file and all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @metalarm10, @miladz68, and @ysv).
ysv
left a comment
There was a problem hiding this comment.
@ysv reviewed 327 files and all commit messages.
Reviewable status:
masihyeganeh
left a comment
There was a problem hiding this comment.
@masihyeganeh reviewed 6 files and all commit messages.
Reviewable status:
ysv
left a comment
There was a problem hiding this comment.
@ysv reviewed 6 files and all commit messages.
Reviewable status:
Description
This pull request upgrades the codebase from version
v6tov7of thegithub.com/tokenize-x/tx-chaindependency throughout the application. It removes all v6-specific upgrade logic and related test files, and introduces a minimal v7 upgrade handler. The main focus is on cleaning up legacy upgrade code and ensuring the application is ready for the new version.Key changes include:
Dependency and Import Updates:
github.com/tokenize-x/tx-chain/v6are updated to usev7across the codebase, including inapp.go, test files, and the module replacement inbuild/go.mod. [1] [2] [3] [4]Upgrade Handler Changes:
appupgradev6.Newtoappupgradev7.Newinapp.go, and a new minimal v7 upgrade handler is implemented inapp/upgrade/v7/upgrade.go. This new handler performs only module migrations without any additional logic. [1] [2]Removal of v6 Upgrade Logic and Tests:
Reviewers checklist:
Authors checklist
This change is