Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Rust/Cosmos]: Move Cosmos blockchain into Rust #3498

Merged
merged 45 commits into from
Nov 16, 2023
Merged

Conversation

satoshiotomakan
Copy link
Collaborator

@satoshiotomakan satoshiotomakan commented Oct 23, 2023

Description

Move all cosmos-sdk based blockchains to Rust.

How to test

Breaking changes

TODO

  • Move Thorchain signing logic to Rust
  • Broadcast StakeAuthorization transaction
  • Remove C++ implementation
  • Move all C++ tests to Rust (at the next PR)
  • Add fuzz tests

Checklist

  • Create pull request as draft initially, unless its complete.
  • Add tests to cover changes as needed.
  • Update documentation as needed.
  • If there is a related Issue, mention it in the description.

If you're adding a new blockchain

  • I have read the guidelines for adding a new blockchain.

* TODO fix Cosmos Protobuf generation
* Rename `EmptyCoinContext` to `TestCoinContext`
* TODO add more tests, add remain Protobuf messages
* Handle `SignDirect` message
* TODO figure out if `StakeAuthorization` serialization can be fixed
* Add tests for WASM Terra and standard WASM contract exec messages
* TODO continue adding tests for WASM exec messages
* TODO: Refactor `TWSigner`
* Add `NativeEvmos` blockchain in C++
* Implement missing TX compiler for Cosmos chains
* TODO: Fix TWAnyAddressData
@satoshiotomakan satoshiotomakan marked this pull request as ready for review November 16, 2023 13:04
include/TrustWalletCore/TWBlockchain.h Show resolved Hide resolved
registry.json Show resolved Hide resolved
@Milerius Milerius merged commit 7119750 into master Nov 16, 2023
13 checks passed
@Milerius Milerius deleted the s/rust-cosmos branch November 16, 2023 18:02
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.

2 participants