Skip to content

Resolve of runtime panic in msgs sendings.#2211

Merged
breathx merged 7 commits intomasterfrom
dn-gas-less-full-sendings-fix
Feb 9, 2023
Merged

Resolve of runtime panic in msgs sendings.#2211
breathx merged 7 commits intomasterfrom
dn-gas-less-full-sendings-fix

Conversation

@breathx
Copy link
Copy Markdown
Member

@breathx breathx commented Feb 8, 2023

Resolves https://github.com/gear-tech/core-audit/issues/22

It's temporary solution just to remove possibility of runtime to panic. Proper one, that doesn't brake or add new invariants will be applied soon.

@breathx breathx added A0-pleasereview PR is ready to be reviewed by the team D2-node Gear Node C0-bug Something isn't working labels Feb 8, 2023
@breathx breathx requested review from gshep and techraed February 8, 2023 19:40
Copy link
Copy Markdown
Member

@gshep gshep left a comment

Choose a reason for hiding this comment

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

LGTM! 💯

@breathx breathx merged commit ea76383 into master Feb 9, 2023
@breathx breathx deleted the dn-gas-less-full-sendings-fix branch February 9, 2023 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A0-pleasereview PR is ready to be reviewed by the team C0-bug Something isn't working D2-node Gear Node

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants