-
Notifications
You must be signed in to change notification settings - Fork 96
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
Update GhoStewardV2.sol #402
Conversation
- Allowed GSM fees to be lowered - Allowed GSM exposure & borrow cap to be changed both ways
|
governance discussions: https://governance.aave.com/t/arfc-gho-stewards/16466/10?u=marczeller |
implementation looking good but can you please update the tests on TestGhoStewardV2.t.sol please?
Also it would be good to have successful tests when lowering the parameters, and reverting when updating a value lower than allowed. |
Please, update |
Closes #401 |
@JoaquinBattilana did a separated PR for tests: #403 |
Merged to a dedicated branch to consolidate all changes and trigger tests |
* Update GhoStewardV2.sol (#402) - Allowed GSM fees to be lowered - Allowed GSM exposure & borrow cap to be changed both ways * fix: Fix certora specs for GhoSteward --------- Co-authored-by: Marc Zeller <[email protected]>
Allowed GSM fees to be lowered
Allowed GSM exposure & borrow cap to be changed both ways