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

Refactor MBT for ICS02 & ICS03 using new modelator Rust API #812

Closed
wants to merge 34 commits into from

Conversation

andrey-kuprianov
Copy link
Contributor

Closes: cosmos/ibc-rs#29

Description

Refactoring of MBT for ICS02 & ICS03 using the new modelator Rust API


For contributor use:

  • Updated the Unreleased section of CHANGELOG.md with the issue.
  • If applicable: Unit tests written, added test to CI.
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Updated relevant documentation (docs/) and code comments.
  • Re-reviewed Files changed in the Github PR explorer.

@andrey-kuprianov andrey-kuprianov added the O: tests Objective: Test more aspect of the relayer label Apr 9, 2021
@andrey-kuprianov andrey-kuprianov self-assigned this Apr 9, 2021
@romac
Copy link
Member

romac commented May 7, 2021

@andrey-kuprianov What's the status on this? Can we expect to merge it sometime or is there much left to do?

@andrey-kuprianov
Copy link
Contributor Author

@romac There is not much to do actually, besides that this depends on the informalsystems/modelator#44 PR being merged, and there are still some bugs preventing this. I will address those in the beginning of the next week.

Sorry for the delay -- there are many other things I need to take care of, unfortunately, so everything progresses a bit slow.

@romac
Copy link
Member

romac commented May 7, 2021

Thanks for the update! I didn't want to imply there was a rush of any kind, just wanted to enquire about the status as I'd like to see that work being put to use for the sake of Hermes :)

@andrey-kuprianov
Copy link
Contributor Author

actually thank you for the update: knowing that this might be useful for Hermes is a very good motivator for me to finish it sooner rather than later;)

@adizere adizere mentioned this pull request Aug 2, 2021
5 tasks
@adizere
Copy link
Member

adizere commented Nov 5, 2021

Closing: deprioritized.

@adizere adizere closed this Nov 5, 2021
@romac romac deleted the andrey/modelator-rust-api branch January 26, 2022 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O: tests Objective: Test more aspect of the relayer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants