Philippe Laferrière

We changed where the height check is (now [here](https://github.com/cosmos/ibc-rs/blob/3a4acfd64d80277808ba0e8cc5ff1c50ca6f7966/crates/ibc/src/core/ics03_connection/handler/conn_open_try.rs#L42) for `conn_open_try`), but we still need to understand the discrepancy (we do `>`, ibc-go does `>=`). This might have something to...

Related: #284

Hmm, I should amend the description. We don't always want to release whatever's in `main` (at least how we do things today); sometimes multiple PRs need to be merged as...

Our requirements are: 1. When a security patch or a new version of ibc-rs with an updated dependency (e.g. tendermint-rs) needs to be released, we want to be able to...

[Related discussion on tendermint-rs](https://github.com/informalsystems/tendermint-rs/issues/1128)

Could you expand on what you mean by a "Store SendPacket interface"?

I believe it's there to enable specific use cases in IBC apps, e.g. [relayer incentivisation](https://github.com/cosmos/ibc/tree/main/spec/app/ics-029-fee-payment). Basically, at a high level, for the token transfer app to be able to "tip"...

Yes, definitely

The problem with this suggested signature is that *all* error variants of `TokenTransferError` would need to have the logs in them, since we always want the logs, no matter the...

But then you're implicitly saying that `CannotMintCoins` is the only viable variant to return, since all other variants will be throwing logs away.