-
Notifications
You must be signed in to change notification settings - Fork 7
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
Stake: using tag v1.0.0 impl #429
Conversation
changed the math back to safe math not only the highlighted places, but in a bunch of others. What's cool is that we can also use checked math when doing |
9f3c8ad
to
b89193f
Compare
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.
👍
62c48bc
to
43bfe3f
Compare
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.
LGTM 👍
some progress
43bfe3f
to
5b67e0d
Compare
this PR removes the stake-rewards contract and brings back the old stake contract implementation