-
Notifications
You must be signed in to change notification settings - Fork 27
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
[SC-989][SC-990] Solidity 0.8.23, OZ 5.0.0 #31
Conversation
zZoMROT
commented
Nov 14, 2023
•
edited
edited
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #31 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 3 3
Lines 99 99
Branches 21 21
=========================================
Hits 99 99 ☔ View full report in Codecov by Sentry. |
Please add gas reports comparison |
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.
Reuse .solcover.js
from here https://github.com/1inch/solidity-utils/blob/master/.solcover.js
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.
With config from solidity-utils
we have Stack Too Deep error
YulException: Cannot swap Variable expr with Slot TMP[eq, 0]: too deep in the stack by 2 slots in [ _6 expr var_account expr expr_2 _11 _10 _9 _8 _7 expr_2186_mpos _5 _4 _3 RET _2 _13 var_i var_account TMP[eq, 0] ]
memoryguard was present.
Done. You can find it in first comment of this PR |