julienrbrt
julienrbrt
> should we merge the readme, the test is fine to remove I think we should add the whole readme https://github.com/cosmos/cosmos-sdk/blob/main/docs/build/building-modules/16-testing.md to mention system tests and remove e2e tests.
ref: https://github.com/cosmos/cosmos-sdk/pull/22278/files#diff-37832b152aeffcb6b1aa5704837f3fa357876b1993277b3657722863192f3f83R123
Wouldn't we able to use the _rest v2_ (post endpoint) server, and proxy query to it? Basically, an http handler that split the request path, and checks the proto registry...
Given that https://github.com/cosmos/cosmos-sdk/pull/22701 did the wiring, closing this as completed. I'll create another issue with the way we wanted to do it
Changing the `upgrade-info.json` is an `x/upgrade` concern. This feature request doesn't seem to be an `x/upgrade` but mainly a Cosmovisor UX improvement. I'd say we actually shouldn't change the `upgrade-info.json`...
I like `add-batch-upgrades` personally, as it isn't something you always need, best to split the concerns 👍 Are you willing to implement it?
Is there any progress on this? That would be useful.
@xujiangyu, let's add the recommended test by @alpe before merging.
@glebarez any plan to merge this?
Is this really useful? Personally I'd rather have the node halt, so you know there's an issue directly, instead of having to continuously fetch for the latest height and see...