Romain Ruetschi
Romain Ruetschi
> Maybe we should be using postMap? That's what I would try as well. I am not sure why we even use `preMap` here.
@thanethomson Will this still be needed now that 0.35 is going to be yanked?
> Given more recent conversations around this event structure, @romac and @adizere does it perhaps make sense to first have a design session around this with the Tendermint Core team...
I thought that we were only planning on releasing the `ibc-relayer-cli` crate as v1, but not the other crates?
> I do not see a case where this log is very useful, so I tend to be of the opinion to remove it from the CLIs. Agreed!
I agree that your proposal is probably the best way forward!
> Cache the parameter (in the ChainHandle?) on startup. You should be able to cache the value on first use in the CachingHandle, backed by a cache with no TTL.
Some code for collating adjacent elements: https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=20c056586af6618d9ad52eda3081574b
Awesome, great work Ali! Before we merge, can we also change `--path` to `--output` to avoid confusion between the path of the config file that's generated and a relay path...
Yep, let's re-open all the PRs in v1.1 milestone