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

Remove tx error sentinel #2962

Open
sug0 opened this issue Mar 27, 2024 · 2 comments
Open

Remove tx error sentinel #2962

sug0 opened this issue Mar 27, 2024 · 2 comments
Labels
breaking-change refactor / code quality wasm
Milestone
Future breaking u...

Comments

@sug0
Copy link
Contributor

sug0 commented Mar 27, 2024 •
edited
Loading

Pass invalid commitment, out-of-gas and other errors back to the host in a fashion similar to #2940, instead of setting a sentinel variable.
This was referenced Mar 27, 2024
Merged
Open
@Fraccaman
Copy link
Member

@sug0 did we do this?

@sug0
Copy link
Contributor Author

sug0 commented Apr 30, 2024

@Fraccaman no, not yet

@sug0 sug0 added the wasm label Jun 3, 2024
@cwgoes cwgoes added this to the Future breaking upgrade milestone Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking-change refactor / code quality wasm
Projects
None yet
Milestone
Future breaking upgrade
Development

No branches or pull requests
3 participants
@Fraccaman @sug0 @cwgoes