Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: cosmos-sdk v0.50 upgrade and multi zone enhancements #27

Merged
merged 8 commits into from
Sep 1, 2023
Merged

refactor: cosmos-sdk v0.50 upgrade and multi zone enhancements #27

merged 8 commits into from
Sep 1, 2023

Conversation

bizk
Copy link
Contributor

@bizk bizk commented Aug 30, 2023

Description

This PR upgrades dependencies to latest cosmos-sdk v0.50.x and also improves project multizone capabilities as type reflection through grpc endpoints.

Work description:

  • changed go.mod name, renamed dependencies on rosetta, adapted methods and uses of cosmos-sdk, upgraded dependencies

  • fixed all code errors

  • improvements over parsing

  • fees and gas parsing

  • remmoved validation on message due to deprecation (it should be validated on the server side)

  • solved issue on missing tx tipper

  • added codec types

  • fixed interface registre addr issue

  • improvements

  • fixed converter tests

  • commented plugin_test.go due to some modules not being available at v0.50 yet

  • improved code, moved parsing functions to utils.go

  • solved comments, fixed lint and added utils.go for parsing

  • fixed comment on cast variables

  • lint comments

  • Commented ibc-go dependdency until upgrade

  • added type assertion error handlinhg

  • updated go.sum

bizk added 4 commits July 4, 2023 12:47
* added plugins feature and reflection features

* fixed minor sdkTx variables issue

* Roll back some changes on config.go

* fixed makefile build command

* improvements over lint

* Added nolint ant changed variable name

* fix .gitworkflow build

* fix .gitworkflow build

* Added makefile build plugin steps

* added nolint on staticcheck on load.go

* added nolint on staticcheck on load.go

* added nolint on staticcheck on load.go

* added nolint on staticcheck on load.go

* gofumpt on project

* skiped a test due to reliance on an experimental feature of cosmos v0.46.0-beta

* build plugin before test

* build plugin before test

* fixed failing test

* fix make test

* added docs

* small refactor on variable names
* added plugins feature and reflection features

* fixed minor sdkTx variables issue

* Roll back some changes on config.go

* fixed makefile build command

* improvements over lint

* Added nolint ant changed variable name

* fix .gitworkflow build

* fix .gitworkflow build

* Added makefile build plugin steps

* added nolint on staticcheck on load.go

* added nolint on staticcheck on load.go

* added nolint on staticcheck on load.go

* added nolint on staticcheck on load.go

* gofumpt on project

* skiped a test due to reliance on an experimental feature of cosmos v0.46.0-beta

* build plugin before test

* build plugin before test

* fixed failing test

* fix make test

* added docs

* small refactor on variable names

* Updated documentation and added --plugin flag

* updated install step on README

* lint fix

* updated docs
@bizk bizk self-assigned this Aug 30, 2023
@bizk bizk added the zondax label Aug 30, 2023
@bizk bizk marked this pull request as draft August 30, 2023 15:25
* changed go.mod name, renamed dependencies on rosetta, adapted methods and uses of cosmos-sdk, upgraded dependencies

* fixed all code errors

* improvements over parsing

* fees and gas parsing

* remmoved validation on message due to deprecation (it should be validated on the server side)

* solved issue on missing tx tipper

* added codec types

* fixed interface registre addr issue

* improvements

* fixed converter tests

* commented plugin_test.go due to some modules not being available at v0.50 yet

* improved code, moved parsing functions to utils.go

* solved comments, fixed lint and added utils.go for parsing

* fixed comment on cast variables

* lint comments

* Commented ibc-go dependdency until upgrade

* added type assertion error handlinhg

* updated go.sum
@bizk bizk marked this pull request as ready for review August 30, 2023 16:09
@bizk bizk requested a review from raynaudoe August 30, 2023 16:15
@bizk bizk changed the title refactor: Multi zone improvements and go mod cleanup refactor: cosmos-sdk v0.50 upgrade and multi zone enhancements Aug 30, 2023
go.mod Outdated
@@ -1,100 +1,124 @@
module cosmossdk.io/tools/rosetta
module cosmossdk.io/rosetta
Copy link
Member

Choose a reason for hiding this comment

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

I think it should be github.com/cosmos/rosetta, the vanity url isn't configured for this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

got it, thx mate!

@bizk bizk requested a review from julienrbrt August 31, 2023 20:21
Copy link
Member

@julienrbrt julienrbrt left a comment

Choose a reason for hiding this comment

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

utACK

@bizk bizk enabled auto-merge (squash) September 1, 2023 14:54
@raynaudoe raynaudoe self-requested a review September 1, 2023 15:03
@bizk bizk merged commit f4f74b4 into cosmos:main Sep 1, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants