-
Notifications
You must be signed in to change notification settings - Fork 391
Rename entry points #817
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
Rename entry points #817
Conversation
webmaster128
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💪
Right, this should not be remaned anymore because it is obsolete. Note in the migrating document is missing indeed. Could you create one for 0.14.0? Let's not bloc this PR with it such that we get the interface changes out for wasmvm and wasmd. |
|
When you get the chance, could you rebase that on main to remove the cranelift CI jobs? |
23f56a0 to
2d17593
Compare
Co-authored-by: Simon Warta <[email protected]>
Closes #807.
HandleMsg->ExecuteMsg.Note: Didn't rename
HandleResponse, because I understand it's being deprecated. I think we should mention that in the migration doc, i.e. to use the newResponsetype instead ofInitResponse/HandleResponse.