-
Notifications
You must be signed in to change notification settings - Fork 213
[WIP] - chore: add complete interfaces #59
base: develop
Are you sure you want to change the base?
Conversation
Side question. How do I pass the commitlint tests? |
@fubuloubu commitlint seems to keep using old commit messages? Specifically "fixed failing tests" and "added complete interfaces". Am I misunderstanding something? |
@tansawit you'll have to amend those commit messages to get it to pass ( |
b3bf512
to
cd55861
Compare
@fubuloubu Thanks! Seems to be working now. Will keep adding to it over the weekend |
@tansawit changed title to WIP to simplify for house keeping |
@tansawit some time has passed since this PR. You are almost there! Can you make the CI checks pass I think we are good to go |
fix commitlint and removed function modifier in interfaces
cd55861
to
078123f
Compare
@fameal apologies. meant to get back to finishing it much sooner. cleaned it up and the tests are now passing :) |
@fubuloubu can we merge? |
A lot of functions are still missing from curve/dforce/maker/yearn are still missing. I can help add those as well if needed but might not get to it until at least the weekends |
Sure, no problem. There's no rush. I'm following up on the PR 💪 |
Fixes: #55
Still a work in progress.
Right now I think I should have the intefaces for Aave and Compound basically complete. One thing I'm wondering that in
aave/AaveToken.sol
, you seem to have madeunderlyingAssetAddress
into a function whereas in the original Aavetokenization/Atoken.sol
,undelyingAssetAddress
is a public variable. Do you want all of the public variables in the contracts to be converted into a function?