-
Notifications
You must be signed in to change notification settings - Fork 32
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: adjust rf and LTV on polygon #613
feat: adjust rf and LTV on polygon #613
Conversation
🌈Test ResultsCompiling 320 files with Solc 0.8.22 Solc 0.8.22 finished in 151.19s Compiler run successful with warnings: Warning (2018): Function state mutability can be restricted to pure --> lib/aave-helpers/src/ProtocolV2TestBase.sol:664:3: | 664 | function _logReserveConfig(ReserveConfig memory config) internal view { | ^ (Relevant source part starts here and spans across multiple lines).2025-02-20T12:16:13.252548Z ERROR cheatcodes: non-empty stderr input=["npx", "@bgd-labs/aave-cli@^1.1.17", "diff-snapshots", "./reports/AaveV2Polygon_AdjustRiskParametersForAaveV2AndV3OnPolygon_20250210_before.json", "./reports/AaveV2Polygon_AdjustRiskParametersForAaveV2AndV3OnPolygon_20250210_after.json", "-o", "./diffs/AaveV2Polygon_AdjustRiskParametersForAaveV2AndV3OnPolygon_20250210_before_AaveV2Polygon_AdjustRiskParametersForAaveV2AndV3OnPolygon_20250210_after.md"] stderr="npm warn exec The following package was not found and will be installed: @bgd-labs/[email protected]\n" Ran 1 test for src/20250210_Multi_AdjustRiskParametersForAaveV2AndV3OnPolygon/AaveV2Polygon_AdjustRiskParametersForAaveV2AndV3OnPolygon_20250210.t.sol:AaveV2Polygon_AdjustRiskParametersForAaveV2AndV3OnPolygon_20250210_Test Suite result: ok. 1 passed; 0 failed; 0 skipped; finished in 16.27s (15.77s CPU time) Ran 1 test suite in 16.28s (16.27s CPU time): 1 tests passed, 0 failed, 0 skipped (1 total tests) Ran 1 test for src/20250210_Multi_AdjustRiskParametersForAaveV2AndV3OnPolygon/AaveV3Polygon_AdjustRiskParametersForAaveV2AndV3OnPolygon_20250210.t.sol:AaveV3Polygon_AdjustRiskParametersForAaveV2AndV3OnPolygon_20250210_Test Suite result: ok. 1 passed; 0 failed; 0 skipped; finished in 2.18s (2.00s CPU time) Ran 1 test suite in 2.19s (2.18s CPU time): 1 tests passed, 0 failed, 0 skipped (1 total tests) |
lib/aave-helpers
Outdated
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 think the lib is not updated to the latest, so the tests still fail
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.
Very weird, I did forge update lib/aave-helpers
.
In the last commit (ddab079) I just ran forge install https://github.com/bgd-labs/aave-helpers
. Maybe a bit hardcore but then it should works
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.
As you told me in PM, with branch=main
it worked like a charm.
Should be good now @brotherlymite
...i_AdjustRiskParametersForAaveV2AndV3OnPolygon/AdjustRiskParametersForAaveV2AndV3OnPolygon.md
Outdated
Show resolved
Hide resolved
...i_AdjustRiskParametersForAaveV2AndV3OnPolygon/AdjustRiskParametersForAaveV2AndV3OnPolygon.md
Outdated
Show resolved
Hide resolved
...i_AdjustRiskParametersForAaveV2AndV3OnPolygon/AdjustRiskParametersForAaveV2AndV3OnPolygon.md
Outdated
Show resolved
Hide resolved
...i_AdjustRiskParametersForAaveV2AndV3OnPolygon/AdjustRiskParametersForAaveV2AndV3OnPolygon.md
Outdated
Show resolved
Hide resolved
…/AdjustRiskParametersForAaveV2AndV3OnPolygon.md Co-authored-by: Harsh Pandey <[email protected]>
…/AdjustRiskParametersForAaveV2AndV3OnPolygon.md Co-authored-by: Harsh Pandey <[email protected]>
…/AdjustRiskParametersForAaveV2AndV3OnPolygon.md Co-authored-by: Harsh Pandey <[email protected]>
* feat: adjust rf and LTV on polygon * feat: set ltv to 0 instead of freeze * ⬆️ bump aave-helpers * forge install: aave-helpers v3.0.0 * ➖ remove aave-helpers * forge install: aave-helpers v3.0.0 * ⬆️ bump aave-helpers to last commit * Update src/20250210_Multi_AdjustRiskParametersForAaveV2AndV3OnPolygon/AdjustRiskParametersForAaveV2AndV3OnPolygon.md Co-authored-by: Harsh Pandey <[email protected]> * Update src/20250210_Multi_AdjustRiskParametersForAaveV2AndV3OnPolygon/AdjustRiskParametersForAaveV2AndV3OnPolygon.md Co-authored-by: Harsh Pandey <[email protected]> * Update src/20250210_Multi_AdjustRiskParametersForAaveV2AndV3OnPolygon/AdjustRiskParametersForAaveV2AndV3OnPolygon.md Co-authored-by: Harsh Pandey <[email protected]> * Update AdjustRiskParametersForAaveV2AndV3OnPolygon.md --------- Co-authored-by: MartinGbz <[email protected]> Co-authored-by: Harsh Pandey <[email protected]>
No description provided.