cosmos / b9-checkers-academy-draft

32 stars 41 forks source link

Opportunity to simplify integration test tutorial #34

Open xavierlepretre opened 2 years ago

xavierlepretre commented 2 years ago

The tutorial bases on ignite 0.17.3, which itself uses tendermint/spm. Adding import "github.com/tendermint/spm/cosmoscmd" and replacing this line by encCdc := cosmoscmd.MakeEncodingConfig(ModuleBasics) should behave the same, if I understood the code correctly.

If my assumption is correct, this change would simplify the tutorial, because app/encoding.go and app/param/proto.go could be omitted.

With the current ignite version, the import has to be changed to "github.com/ignite/cli/ignite/pkg/cosmoscmd"

_Originally posted by @l-wegner in https://github.com/cosmos/b9-checkers-academy-draft/pull/12#discussion_r943615706_