-
Notifications
You must be signed in to change notification settings - Fork 4
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: add all scripts from multisig to governance #18
Conversation
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 prefer to have only a few script that works and then adapt to all the ones in the multisig
@@ -0,0 +1,59 @@ | |||
pragma solidity ^0.8.19; | |||
|
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 encountered that in the angle-multisig, maybe better to create interfaces in respective repo to not copy paste evrywhere
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #18 +/- ##
=======================================
Coverage 85.49% 85.49%
=======================================
Files 7 7
Lines 255 255
=======================================
Hits 218 218
Misses 37 37 ☔ View full report in Codecov by Sentry. |
b968164
to
3af5b78
Compare
# if [[ $execute == "yes" ]]; then | ||
# forge script scripts/interaction/Propose.s.sol:Propose --fork-url $mainnet_uri --broadcast | ||
# fi |
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.
don't forget to uncomment this
@@ -204,6 +204,7 @@ contract SimulationSetup is Test { | |||
HELPERS | |||
//////////////////////////////////////////////////////////////////////////////////////////////////////////////////*/ | |||
|
|||
// TODO USE IT |
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.
remove this forgotten comment
No description provided.