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.

Sign up for GitHub

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

Jump to bottom

Upgrade CosmWasm to wasmd 0.18 #863

Merged
merged 4 commits into from
Aug 4, 2021
Merged

Upgrade CosmWasm to wasmd 0.18 #863

merged 4 commits into from
Aug 4, 2021

Conversation

webmaster128
Copy link
Member

@webmaster128 webmaster128 commented Jul 29, 2021 •
edited

Closes #829

  • Upgrade generated proto types
  • Use v1 proto types
  • Update backend
  • Make everything pass
  • Adapt version compatibility table
  • Add CHANGELOG entries

@webmaster128 webmaster128 marked this pull request as ready for review August 2, 2021 12:04
@ethanfrey
Copy link
Contributor

Looks good to me. Do you need anything else from the wasmd side?

ethanfrey
ethanfrey approved these changes Aug 3, 2021
View reviewed changes
Copy link
Contributor

@ethanfrey ethanfrey left a comment

Choose a reason for hiding this comment

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

Good stuff. Let's go!

packages/cosmwasm-stargate/src/aminotypes.spec.ts Show resolved Hide resolved
packages/cosmwasm-stargate/src/aminotypes.ts Show resolved Hide resolved
packages/cosmwasm-stargate/src/queries/wasm.spec.ts Show resolved Hide resolved
packages/cosmwasm-stargate/src/testutils.spec.ts Show resolved Hide resolved
scripts/wasmd/contracts/checksums.sha256
3e3ea92c4ec35d90aa0bef344bdb0a94939120fce57265d5ff81bf8e8caec7e9 cw3_fixed_multisig.wasm
a32acdcfe15a2b3c8ba6963cf1e4ab63347725cc35a0f2434696dd492d63fb5f hackatom.wasm
089598e3cab468aff26233bc7363c713529124a0f7fd50320461cb58881a2da7 ibc_reflect.wasm
37f868eda7a723a769f6ff77cc27435455c3cff7b14481a114895d3d93cf8326 cw1_subkeys.wasm
Copy link
Contributor

Choose a reason for hiding this comment

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

This is based on v0.8.0-rc1 and v0.16.0-rc5, right?

Do you need a final cosmwasm-plus release before you can cut cosmjs? We are not changing the interfaces of those contracts since the versions you use

Copy link
Member Author

Choose a reason for hiding this comment

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

Right. I don't need anything else from cosmwasm-plus. Those contracts are not used right now.

@webmaster128 webmaster128 merged commit aabca4f into main Aug 4, 2021
@webmaster128 webmaster128 deleted the upgrade-to-wasmd-0.18 branch August 4, 2021 14:54
@spacepotahto spacepotahto mentioned this pull request Sep 9, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ethanfrey ethanfrey ethanfrey approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add support for CosmWasm 0.16/wasmd 0.18
2 participants
@webmaster128 @ethanfrey