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

[WIP] Wasmd upgrade #74

Merged
merged 6 commits into from
Aug 4, 2021
Merged

[WIP] Wasmd upgrade #74

merged 6 commits into from
Aug 4, 2021

Conversation

alpe
Copy link
Contributor

@alpe alpe commented Jul 5, 2021

Resolves #78

Integrates with wasmd master

Left to do:

  • Point to a wasmd version in master
  • replace manual built PoE contracts with a proper version
  • Point to wasmd release

😆 just noticed that I unintentionally had named this branch integrade_ instead of integrate_. That's proper commitment...

@ethanfrey
Copy link
Contributor

PoE contracts were built locally from a custom branch to be compatible with wasmvminterface_version_6

This comment is no longer true, right? They come from v0.3.0-rc1 with interface_version_7, correct?

Point to wasmd release

If you cut wasmd v0.18.0-rc2 off the current master, then we could point this and merge it. I would prefer to merge such update quickly so other PRs can build off the latest version. We can always make another PR to bump things from rc2/rc5 to final release, but it should be very light.

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.

Looks good. Let's tag a wasmd release and merge this.

@alpe alpe marked this pull request as ready for review August 4, 2021 15:13
@alpe
Copy link
Contributor Author

alpe commented Aug 4, 2021

Upgrade wasmd version to v0.18.0-rc2

@alpe alpe merged commit 51dd238 into main Aug 4, 2021
@alpe alpe deleted the integrade_wasmvm15_early branch August 4, 2021 15:16
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.

Upgrade to Wasmd 0.18
2 participants