Skip to content

refactor(coredaos): migrate away from atomone's x/gov wrapper#353

Merged
giunatale merged 3 commits into
mainfrom
giunatale/refactor/coredaos-gov
Jun 29, 2026
Merged

refactor(coredaos): migrate away from atomone's x/gov wrapper#353
giunatale merged 3 commits into
mainfrom
giunatale/refactor/coredaos-gov

Conversation

@giunatale

Copy link
Copy Markdown
Collaborator

This PR removes the dependency on atomone x/gov wrapper from x/coredaos in favor of using the atomone-sdk x/gov.

This work alongside atomone-hub/atomone-sdk#106 is a prerequisite for a future-coming PR that does some further cleaning/refactoring.

@tbruyelle tbruyelle left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Having to deal direclty with the gov keeper fields like keeper.Proposals is a bit annoying as it forbids the usage of a local interface. But that's out of the scope of this PR.
LGTM

@giunatale

Copy link
Copy Markdown
Collaborator Author

Having to deal direclty with the gov keeper fields like keeper.Proposals is a bit annoying as it forbids the usage of a local interface. But that's out of the scope of this PR.
LGTM

nothing to do here unfortunately because of collections, using the keeper directly is unavoidable. The methods the previous interface was using are simply gone

@giunatale giunatale merged commit 25c8953 into main Jun 29, 2026
14 checks passed
@giunatale giunatale deleted the giunatale/refactor/coredaos-gov branch June 29, 2026 10:43
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 this pull request may close these issues.

2 participants