-
Notifications
You must be signed in to change notification settings - Fork 146
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
feat: update some contracts to use solidity 0.8 #725
Conversation
Socket Security ReportDependency issues detected. If you merge this pull request, you will not be alerted to the instances of these issues again. 📜 New install scripts detectedA dependency change in this PR is introducing new install scripts to your install step.
🫣 Native codeContains native code which could be a vector to obscure malicious code, and generally decrease the likelihood of reproducible or reliable installs.
😵💫 Bin script confusionThis package has multiple bin scripts with the same name. This can cause non-deterministic behavior when installing or could be a sign of a supply chain attack.
Socket.dev scan summary
Bot CommandsTo ignore an alert, reply with a comment starting with
Powered by socket.dev |
Codecov ReportBase: 91.56% // Head: 91.54% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## pcv/l2-linear-rewards #725 +/- ##
=========================================================
- Coverage 91.56% 91.54% -0.03%
=========================================================
Files 42 42
Lines 2003 1998 -5
Branches 350 350
=========================================================
- Hits 1834 1829 -5
Misses 169 169
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
…ecked fix: use unchecked math for address aliasing (C4 M-289 from #699)
experimental / WIP, take this with a bucketful of salt, might not work, etc