Skip to content

build(deps)!: bump cosmos sdk to v0.50.x [DEV-4827]#874

Merged
Eengineer1 merged 17 commits intodevelopfrom
akhil/v0.50_bump
May 14, 2025
Merged

build(deps)!: bump cosmos sdk to v0.50.x [DEV-4827]#874
Eengineer1 merged 17 commits intodevelopfrom
akhil/v0.50_bump

Conversation

@akhilkumarpilli
Copy link
Copy Markdown
Contributor

closes: #871

@Eengineer1 Eengineer1 self-requested a review April 8, 2025 15:18
Comment thread go.mod
@akhilkumarpilli akhilkumarpilli changed the title build(deps)!: bump cosmos sdk to v0.50.x WIP: build(deps)!: bump cosmos sdk to v0.50.x Apr 9, 2025
@akhilkumarpilli akhilkumarpilli changed the title WIP: build(deps)!: bump cosmos sdk to v0.50.x build(deps)!: bump cosmos sdk to v0.50.x Apr 15, 2025
@akhilkumarpilli akhilkumarpilli requested review from Eengineer1 and removed request for Eengineer1 April 15, 2025 10:51
Comment thread api/go.mod
Comment thread go.mod Outdated
Comment thread api/cheqd/did/v2/fee.pulsar.go
Copy link
Copy Markdown
Member

@Eengineer1 Eengineer1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Other than the pending streaming services registration, app wiring looks good + ready to go, when tests are restored.

Comment thread app/app.go
@Eengineer1
Copy link
Copy Markdown
Member

Let's focus on resolving the build errors, before moving on with unit, integration + upgrade tests in that order.

@tweeddalex tweeddalex changed the title build(deps)!: bump cosmos sdk to v0.50.x build(deps)!: bump cosmos sdk to v0.50.x [DEV-4827] May 6, 2025
@ankurdotb
Copy link
Copy Markdown
Contributor

Task linked: DEV-4827 Bump Cosmos SDK to v0.50

* wip: upgrade resource module

* fix typo

* fix

* update test setup

* change types

* wip

* add migrations

* remove validate basic

* fix

* fix test setup

* fix

* feat!: upgrade did module to v0.50.x (#878)

* wip

* wip

* wip

* wip

* feat(params): add migrator support to did module

* feat: add autocli support

* wip

* fix

* wip

* chore: remove comment

* chore: add migrate

* chore: add migration in module.go

* feat: add SetParams function

* go.mod

* fix

* setconfig

* fix proto annotations

* feat: change functions

* chore: fix param_change_proposal tests

* chore: remove comment

* fix keeper tests

* feat: implement customgetsigner

* wip

* chore: added query params func and updated the params key

* add custom signers

* fix key

* fix

* chore: add params to autocli

* add params

* fix autocli

* fix typo

* fix

* fix

* fix lint and errors

* update golangci lint version in gha

* fix tests setup

* fix almost failing unit tests

* fix lint and other changes

* revert integration test changes

* WIP: fee abs tests fix

* add resource params query and fix fee abs test

* fix few integration tests and update swagger

* fixed insufficient funds test issue

* WIP: fix query proposal test and address comments

* fix faiing integration test

* fix unit test

* WIP: setup v4 upgrade tests

* fix upgrade migrations issues

* fix post upgrade tests and wip integration tests after upgrade

* fix pre test

* fix post tests

* fix all tests

* fix lint

---------

Co-authored-by: Vishal Potpelliwar <71565171+vishal-kanna@users.noreply.github.com>
Co-authored-by: vishal <kannavish329@gmail.com>
Co-authored-by: akhilkumarpilli <akhilkumar7947@gmail.com>
Co-authored-by: Tasos Derisiotis <50984242+Eengineer1@users.noreply.github.com>
@Eengineer1 Eengineer1 marked this pull request as ready for review May 14, 2025 14:25
Copy link
Copy Markdown
Member

@Eengineer1 Eengineer1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

Signing off base PR as well 👍.

@Eengineer1 Eengineer1 merged commit c7d1235 into develop May 14, 2025
26 checks passed
@Eengineer1 Eengineer1 deleted the akhil/v0.50_bump branch May 14, 2025 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants