-
Notifications
You must be signed in to change notification settings - Fork 0
feat: simrevert type #98
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
base: dylan/chore-deps-sdk-main
Are you sure you want to change the base?
Conversation
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
0a3a830
to
0e8eda5
Compare
0e8eda5
to
1fc9892
Compare
f516bc0
to
91b328f
Compare
2c09604
to
a96a88f
Compare
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.
lgtm
/// Incorrect host block error | ||
IncorrectHostBlock, | ||
/// Bad signature error | ||
BadSignature, |
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.
I assume we're not preserving the data in tihs error since it's probably not worth it do so anyway
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.
it's an error kind not an error itself. it's just a memoization of the type of the actual error data stored elsewhere
adds a revert error convenience type to clean up some code and encapsulate our error munging