cli icon indicating copy to clipboard operation
cli copied to clipboard

Cosmos SDK `v0.46.0` upgrade tracking issue

Open aljo242 opened this issue 3 years ago • 9 comments

This issue can track our progress of upgrading ignite to use the Cosmos-SDK v0.46.0.

Checklist

  • [X] https://github.com/tendermint/spn/pull/914
  • [X] Update go.mod to use updated cosmos-sdk, spn, and ibc-go
  • [X] change all imports starting with github.com/cosmos/ibc-go/v3 to github.com/cosmos/ibc-go/v5
  • [X] update app template according to new SDK and test with ignite
  • [X] Test tutorials after all finished - cc @tbruyelle

Post-release checklist

  • [ ] Update CHANGELOG
  • [ ] Prepare for new release
  • [ ] Update documentation, and compatibility table in the readme @fadeev

aljo242 avatar Jul 28 '22 14:07 aljo242

Feel free to add to this

aljo242 avatar Jul 28 '22 14:07 aljo242

cc @nooomski @marbar3778

ilgooz avatar Jul 28 '22 14:07 ilgooz

Feel free to add to this

I don't have anything particular to add. I think

update app template according to new SDK and test with ignite

Can be separated for all scaffolding commands to work on them in parallel, this could be the complex part of the PR

lumtis avatar Jul 28 '22 14:07 lumtis

May we close https://github.com/ignite/cli/issues/2156 @ilgooz ?

lumtis avatar Jul 28 '22 14:07 lumtis

Hey, I created this PR which is using feat/sdk-ibc-upgrade as the base branch.

I propose to target feat/sdk-ibc-upgrade as the base branch for the future PRs as well and once all of them are merged, we can extensively test that branch before merging it to develop.

This approach could also give us flexibility if we want to make another release from develop while working on this feature in feat/sdk-ibc-upgrade.

ilgooz avatar Jul 29 '22 08:07 ilgooz

I can work on the documentation for this.

fadeev avatar Jul 29 '22 13:07 fadeev

@fadeev That would be great! 🙏

ilgooz avatar Jul 29 '22 13:07 ilgooz

Remaining work on #2677 is to modify the templates and proto-gen

aljo242 avatar Jul 29 '22 15:07 aljo242

Since @tbruyelle is vacationing, @jeronimoalbi would you mind taking the part related to running the tutorials and migrating them if there is a need for it?

ilgooz avatar Aug 04 '22 08:08 ilgooz