Skip to content
This repository has been archived by the owner on Nov 22, 2021. It is now read-only.

[WIP] - chore: add complete interfaces #59

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

tansawit
Copy link

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 made underlyingAssetAddress into a function whereas in the original Aave tokenization/Atoken.sol, undelyingAssetAddress is a public variable. Do you want all of the public variables in the contracts to be converted into a function?

@tansawit
Copy link
Author

Side question. How do I pass the commitlint tests?

@fubuloubu
Copy link
Member

@tansawit
Copy link
Author

@fubuloubu commitlint seems to keep using old commit messages? Specifically "fixed failing tests" and "added complete interfaces". Am I misunderstanding something?

@fubuloubu
Copy link
Member

@tansawit you'll have to amend those commit messages to get it to pass (git rebase -i develop)

@tansawit
Copy link
Author

tansawit commented Oct 29, 2020

@fubuloubu Thanks! Seems to be working now. Will keep adding to it over the weekend

@milkyklim milkyklim changed the title added complete interfaces chore: add complete interfaces Nov 6, 2020
@storming0x storming0x changed the title chore: add complete interfaces [WIP] - chore: add complete interfaces Nov 6, 2020
@storming0x
Copy link

@tansawit changed title to WIP to simplify for house keeping

@saltyfacu
Copy link

@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

@tansawit
Copy link
Author

@fameal apologies. meant to get back to finishing it much sooner. cleaned it up and the tests are now passing :)

@saltyfacu
Copy link

@fubuloubu can we merge?

@tansawit
Copy link
Author

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

@saltyfacu
Copy link

Sure, no problem. There's no rush. I'm following up on the PR 💪

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add complete interfaces for everything
4 participants