Skip to content

Replace tx-tools references#3

Merged
TxCorpi0x merged 6 commits into
masterfrom
mehdi/tx-tools-replace
Oct 8, 2025
Merged

Replace tx-tools references#3
TxCorpi0x merged 6 commits into
masterfrom
mehdi/tx-tools-replace

Conversation

@TxCorpi0x

Copy link
Copy Markdown
Contributor

Description

This pull request migrates the codebase from using the CoreumFoundation/coreum-tools library to the new tokenize-x/tx-tools library. The migration affects both build scripts and integration tests, ensuring all utility imports now reference the new toolkit. Additionally, the dependency on coreum-tools is removed from all go.mod files, and the corresponding package references are updated. The pull request also updates the IBC hooks module to use the tokenize-x fork. These changes improve maintainability and ensure consistency with the latest tooling from the tokenize-x organization.

Dependency migration and updates:

  • Replaced all imports of github.com/CoreumFoundation/coreum-tools with github.com/tokenize-x/tx-tools across build scripts and integration test files, updating utility functions such as must, libexec, and retry to use the new library.

  • Removed github.com/CoreumFoundation/coreum-tools from all relevant go.mod files and added github.com/tokenize-x/tx-tools as a new dependency.

IBC module update:

  • Updated the IBC hooks module to use the github.com/tokenize-x/ibc-apps/modules/ibc-hooks/v8 fork instead of the CoreumFoundation version in go.work and go.work.sum.

Build system dependency pinning:

  • Updated the pinned versions of github.com/tokenize-x/crust and related modules in build/go.mod to newer revisions.

Go module checksums:

  • Added new checksums for the tokenize-x dependencies and the removed coreum-tools in go.work.sum to ensure reproducible builds.

Reviewers checklist:

  • Try to write more meaningful comments with clear actions to be taken.
  • Nit-picking should be unblocking. Focus on core issues.

Authors checklist

  • Provide a concise and meaningful description
  • Review the code yourself first, before making the PR.
  • Annotate your PR in places that require explanation.
  • Think and try to split the PR to smaller PR if it is big.

@TxCorpi0x TxCorpi0x requested review from alinetskyi and ysv and removed request for alinetskyi and ysv October 8, 2025 07:13
@TxCorpi0x TxCorpi0x force-pushed the mehdi/tx-tools-replace branch 4 times, most recently from c2d6181 to d82a019 Compare October 8, 2025 14:26
@TxCorpi0x TxCorpi0x force-pushed the mehdi/tx-tools-replace branch from d82a019 to a97e04e Compare October 8, 2025 14:29
@TxCorpi0x TxCorpi0x force-pushed the mehdi/tx-tools-replace branch from 898d06d to fc0a2c4 Compare October 8, 2025 14:57
@TxCorpi0x TxCorpi0x force-pushed the mehdi/tx-tools-replace branch from 97da847 to 36cddc6 Compare October 8, 2025 15:30
@TxCorpi0x TxCorpi0x merged commit cdb7ab3 into master Oct 8, 2025
8 checks passed
@TxCorpi0x TxCorpi0x deleted the mehdi/tx-tools-replace branch October 9, 2025 14:24
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.

3 participants