-
Notifications
You must be signed in to change notification settings - Fork 48
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: disallow zero startTime in dynamic and tranched #852
Conversation
29123e3
to
bb0566e
Compare
bb0566e
to
2309ea1
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.
Thanks for the PR Andrei. I have reviewed it and left some comments below:
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.
good job
test: when start time is zero in create function from dynamic and tranched test: fuzz start time starting from 1 test: move common invariants in Lockup test: remove no longer needed invariants chore: update Precompiles
2309ea1
to
16aea10
Compare
@smol-ninja I have updated the branch with your feedback, does it look good? |
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.
Perfect. Looks good.
test: when start time is zero in create function from dynamic and tranched test: fuzz start time starting from 1 test: move common invariants in Lockup test: remove no longer needed invariants chore: update Precompiles
test: when start time is zero in create function from dynamic and tranched test: fuzz start time starting from 1 test: move common invariants in Lockup test: remove no longer needed invariants chore: update Precompiles
Addresses #851