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: make cosmwasm_vm::check_wasm public #82

Closed
wants to merge 1 commit into from
Closed

feat: make cosmwasm_vm::check_wasm public #82

wants to merge 1 commit into from

Conversation

loloicci
Copy link
Contributor

@loloicci loloicci commented Mar 4, 2021

Closes: #81

Description

make cosmwasm_vm::check_wasm public to use it from cosmwasm-simulator

Motivation and context

see #81

Checklist:

  • I followed the contributing guidelines.
  • I have updated the documentation accordingly. (Not Needed)
  • I have added tests to cover my changes. (Not Needed)

@loloicci loloicci self-assigned this Mar 4, 2021
@whylee259
Copy link
Contributor

Is this needed for store time error detection?

@loloicci
Copy link
Contributor Author

loloicci commented Mar 9, 2021

Is this needed for store time error detection?

As far as I checked, this is not needed for it. I'm now researching what this function really does.

@loloicci
Copy link
Contributor Author

loloicci commented Mar 17, 2021

We need the function for the simulator and I added the report to #81

@loloicci
Copy link
Contributor Author

closed by #87

@loloicci loloicci closed this Mar 22, 2021
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.

Make public check_wasm
2 participants