Philippe Laferrière
Philippe Laferrière
Is now a good time for me to go for this one? It could create some pretty bad merge conflicts if other big PRs are currently being worked on.
Let's wait for #2300 to be addressed too
We also need this feature. Would you now be able to share an ETA, by any chance? :)
I second this. We really need this feature. Currently, we're hacking it by making the RUN script sleep forever once done, and we put the cleanup in the shutdown script.
Relevant [comment](https://github.com/informalsystems/ibc-rs/pull/2200#issuecomment-1129214839) by @ancazamfir.
Two minor thoughts: 1. I think we should use `ibc-rs` for the modules, and have a statement in the README that redirects to the hermes repo and explains the transition...
Running `cargo update` updated the culprit: `url` from `1.7.0` to `1.7.2`.
The following statement in ICS-3 should also be fixed. > At the moment, the contents of this version bytestring are opaque to the IBC core protocol. In the future, it...
>Hmm, but even if height 0 is valid it would still be sensible to error if it was passed as the timeoutHeight correct? I would personally wouldn't error on height...