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.

Sign up for GitHub

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

Jump to bottom

Refactor Light Client interface to better support IBC-rs and ICS07 #956

Closed
Tracked by #958
thanethomson opened this issue Aug 25, 2021 · 0 comments · Fixed by #960
Closed
Tracked by #958

Refactor Light Client interface to better support IBC-rs and ICS07 #956

thanethomson opened this issue Aug 25, 2021 · 0 comments · Fixed by #960
Labels
enhancement New feature or request ibc Required by IBC light-client Issues/features which involve the light client

Comments

@thanethomson
Copy link
Contributor

As discussed in today's tendermint-rs call, right now there's some ongoing work in ibc-rs (see informalsystems/hermes#1252) that needs to make use of some of the code from the tendermint-light-client crate, but the interfaces currently exposed don't really work well for use from ibc-rs.

For example, the VerificationPredicates trait makes use of LightBlocks as parameters, but ibc-rs doesn't make use of the same LightBlock constructs.

What's the definition of "done" for this issue?

When the Light Client API's been refactored in such a way that we can use its code from informalsystems/hermes#1252
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request ibc Required by IBC light-client Issues/features which involve the light client
Projects
None yet
Milestone
No milestone
1 participant
@thanethomson