Skip to content

Use published TX RUST crates#112

Merged
TxCorpi0x merged 2 commits into
masterfrom
mehdi/use-published-tx-rust-crates
Apr 1, 2026
Merged

Use published TX RUST crates#112
TxCorpi0x merged 2 commits into
masterfrom
mehdi/use-published-tx-rust-crates

Conversation

@TxCorpi0x

@TxCorpi0x TxCorpi0x commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Description

This pull request updates the way the tx-wasm-sdk dependency is specified in several Cargo.toml files. Instead of referencing the SDK from a specific Git repository and tag, it now uses the published version from crates.io. This change improves reproducibility and simplifies dependency management.

Dependency management improvements:

  • Changed the tx-wasm-sdk dependency from a GitHub repository reference to a crates.io version (1.3.0) in integration-tests/contracts/modules/Cargo.toml.
  • Updated the tx-wasm-sdk dependency to use version 1.3.0 from crates.io in x/asset/ft/keeper/test-contracts/asset-extension/Cargo.toml.
  • Updated the tx-wasm-sdk dependency to use version 1.3.0 from crates.io in x/dex/keeper/test-contracts/dex-reentrancy-poc/Cargo.toml.

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.

This change is Reviewable

@TxCorpi0x TxCorpi0x requested a review from a team as a code owner March 27, 2026 09:06
@TxCorpi0x TxCorpi0x requested review from akhlopiachyi, masihyeganeh, metalarm10, miladz68, tx-development and ysv and removed request for a team March 27, 2026 09:06

@metalarm10 metalarm10 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@metalarm10 reviewed 6 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on akhlopiachyi, masihyeganeh, miladz68, tx-development, and ysv).

@ysv ysv left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@ysv reviewed 6 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on akhlopiachyi, masihyeganeh, miladz68, and tx-development).

@TxCorpi0x TxCorpi0x merged commit 03c2226 into master Apr 1, 2026
21 of 22 checks passed
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