You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The gm hermes config command overwrites entirely the config file of Hermes. This is generally not a problem, except we recently changed how the global section of Hermes config looks like, and gm needs to be updated to reflect these recent changes.
Concretely, it used to be that the strategy field in the global section of the Hermes config was a String with the default value of naive (and was even ignored):
…s#1008)
* gm hermes config preserves the global section; fixesinformalsystems#1007.
* Moved parameter loading to load_config
* Added defaults for strategy and log_level
Crate
gm
Summary of Bug
The
gm hermes config
command overwrites entirely the config file of Hermes. This is generally not a problem, except we recently changed how theglobal
section of Hermes config looks like, andgm
needs to be updated to reflect these recent changes.Concretely, it used to be that the
strategy
field in theglobal
section of the Hermes config was a String with the default value ofnaive
(and was even ignored):https://github.com/informalsystems/ibc-rs/blob/d9ce34bb961d5750366dbc970fd116a638424d07/config.toml#L1-L2
With PR #924 we changed the valid options for the
strategy
field to be different:https://github.com/informalsystems/ibc-rs/blob/7c0418e3f6adf270a2a91f19b39a0141234c7545/config.toml#L1-L5
gm v0.0.1
will writenaive
in this field, which Hermes will not recognize.Version
7c0418e
Steps to Reproduce
Acceptance Criteria
gm hermes config
builds a config file that Hermes can parse correctly;gm hermes config
should leave theglobal
section of the config untouched.For Admin Use
The text was updated successfully, but these errors were encountered: