Skip to content

Conversation

@ngyam
Copy link
Contributor

@ngyam ngyam commented Mar 18, 2019

WIP do not merge
@hai-ko @CJentzsch

@ngyam ngyam self-assigned this Mar 18, 2019
@ngyam ngyam force-pushed the feat/reward_contract branch 2 times, most recently from 8c030cb to 6df8ac6 Compare March 24, 2019 14:45
- tests + mock contracts included
@ngyam ngyam force-pushed the feat/reward_contract branch from 6df8ac6 to 8606b3d Compare March 24, 2019 14:47
ngyam added 2 commits March 27, 2019 10:37
- natspec added
- tests cleaned: more descriptive variable names
- S curve + tests added
- some existing tests reworked/fixed
- testing on concrete error messages added
- eternal storage/proxy removed
Copy link

@jbentke jbentke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sauber

@ngyam ngyam requested review from CJentzsch and hai-ko April 1, 2019 14:08
- getters are also removed cause were unnecessary afterwards
@ngyam
Copy link
Contributor Author

ngyam commented Apr 2, 2019

done @hai-ko

@ngyam ngyam merged commit 80893ad into master Apr 2, 2019
@ngyam ngyam deleted the feat/reward_contract branch April 3, 2019 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants