-
Notifications
You must be signed in to change notification settings - Fork 131
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
Handle wrapped errors during storage iteration #2488
Conversation
Cadence Benchstat comparisonThis branch with compared with the base branch onflow:master commit ad21199 Collapsed results for better readability
|
Codecov Report
@@ Coverage Diff @@
## master #2488 +/- ##
=======================================
Coverage 78.28% 78.28%
=======================================
Files 327 327
Lines 72778 72783 +5
=======================================
+ Hits 56972 56979 +7
+ Misses 13710 13708 -2
Partials 2096 2096
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Also added a FVM test: onflow/flow-go@9e4b9a6, to verify the behavior after the integration. Will open a PR to flow-go once a cadence version is released with this fix. |
Closes #2200
Description
Unwrap the returned error and check for the error kind.
master
branchFiles changed
in the Github PR explorer