AlianBenabdallah

Results 17 comments of AlianBenabdallah

I wonder how a `gm reboot` would differ from `gm reset` which is currently implemented. So far, I've been using an alias which does : `gm stop && gm reset...

> It doesn't seem like the CLI parameter names for the `hermes config auto` command adhere to [ADR 010](https://github.com/informalsystems/ibc-rs/blob/master/docs/architecture/adr-010-unified-cli-arguments-hermes.md). I will rename every parameter. I also think `key-names` could be...

I added `path.rs` which contains structs and functions to fetch [IBC data](https://github.com/cosmos/chain-registry/tree/master/_IBC) from the chain repo. Now I need to modify the API in order to incorporate this change. I'll...

> > keys is a required argument. It populates key_name in ChainConfig. > > Discussing with Luca today and we're wondering why would the keys option be required? Hermes could...

Maybe I should try to remove `ChainConfig` dependencies from the `chain-registry` crate.

> > keys is a required argument. It populates key_name in ChainConfig. > > Discussing with Luca today and we're wondering why would the keys option be required? Hermes could...

> I see how they are useful here for providing the corresponding keys in the same order, but perhaps we could improve the ux a little bit in this case...

> 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...

In order to run a relayer on a production channel, it is necessary to have access to a websocket interface. I think that the nodes in the chain-registry disabled this...