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

feat: upgrade sdk to v0.47.x #6413

Merged
merged 216 commits into from
Nov 9, 2023
Merged

feat: upgrade sdk to v0.47.x #6413

merged 216 commits into from
Nov 9, 2023

Conversation

PaddyMc
Copy link
Collaborator

@PaddyMc PaddyMc commented Sep 15, 2023

Closes: #XXX

What is the purpose of the change

@github-actions
Copy link
Contributor

github-actions bot commented Sep 15, 2023

Important Notice

This PR includes modifications to the tests/e2e/initialization module.
Please follow the instructions below:

  1. Backport these changes to the previous Osmosis version's branch.
  2. Run the script inside a Docker container to update genesis and configs for pre-upgrade Osmosis.
  3. Merge the backported changes.
  4. The image will be built and uploaded to Docker Hub here.
  5. Grab the latest image and update it in the PR to the main branch replacing the previousVersionInitTag in the osmosis/tests/e2e/containers/config.go

Please let us know if you need any help.

@@ -3,7 +3,7 @@ package osmosis.gamm.poolmodels.balancer.v1beta1;

import "gogoproto/gogo.proto";
import "amino/amino.proto";
import "osmosis/gamm/pool-models/balancer/balancerPool.proto";
import "osmosis/gamm/v1beta1/balancerPool.proto";
Copy link
Member

Choose a reason for hiding this comment

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

why?

Copy link
Member

Choose a reason for hiding this comment

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

same here #6413 (comment)

@czarcas7ic czarcas7ic marked this pull request as ready for review November 6, 2023 19:58
@czarcas7ic czarcas7ic closed this Nov 8, 2023
@czarcas7ic czarcas7ic reopened this Nov 8, 2023
@czarcas7ic czarcas7ic closed this Nov 8, 2023
@czarcas7ic czarcas7ic reopened this Nov 8, 2023
Copy link
Member

@ValarDragon ValarDragon left a comment

Choose a reason for hiding this comment

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

unforkening!

@czarcas7ic czarcas7ic merged commit 5494ad8 into main Nov 9, 2023
1 check passed
@czarcas7ic czarcas7ic deleted the chore/update-to-v0.47.x branch November 9, 2023 17:01
@github-actions github-actions bot mentioned this pull request May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants