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

Cleaner juno/app #245

Closed
Anmol1696 opened this issue Aug 5, 2022 · 4 comments · Fixed by #366
Closed

Cleaner juno/app #245

Anmol1696 opened this issue Aug 5, 2022 · 4 comments · Fixed by #366

Comments

@Anmol1696
Copy link

Anmol1696 commented Aug 5, 2022

Overview

Create app/upgrades dir to have all the upgrades in one place with versions and abstract out all keepers to app/keepers to make the the app/app.go cleaner and more maintainable.

Problem Definition

app/app.go is too big file.

Proposal

split app/app.go into app.go, upgrade/, keepers/ , module.go

ref:
osmosis/app
provenance/app

@faddat
Copy link
Contributor

faddat commented Aug 7, 2022

Hi there! I really liked your PR for the hub, this one:

I have been in seoul and unable to give it a proper review.

Are the osmo and provenance styles different?

Point blank: I agree with you.

@sascha1337
Copy link

sascha1337 commented Aug 7, 2022

@faddat - refactor useful? might be some good practice to get warm.

regarding the git workflow here - already made a decision on how to get started?
( the pull / branch strategy we should go for )

cosmos/gaia#1580 (review)

@Anmol1696
Copy link
Author

Anmol1696 commented Aug 8, 2022

@faddat provenance style is a little different in the sense they define there upgrades as a map with values as the upgrade function like this, then they have functions to deal with this map and create handlers out of this.

In osmo this is more modular and we create a package itself per upgrade version, instead of a key value in map.

I prefer the osmo way since it is cleaner and will be able to do alot more, includig handling forks better as well.

PS. Would like to contribute to this, once we agree on the approach, i dont mind raising a PR and then have discussion then.

@faddat
Copy link
Contributor

faddat commented Aug 15, 2022

I'm very happy ot use the osmo style :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants