-
Notifications
You must be signed in to change notification settings - Fork 29
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
simulate actions of vote.py in review_votes.py #1531
Conversation
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, but maybe wait until there is a payload that we can run on this branch to test it properly?
can you add |
… reports) + reformat JSON
vLAURA Votes ReviewCSV file: Allocation Check
Snapshot Votes Check
Vote PreparationSuccessfully simulated vote preparation ✅ Vote Summary
✅ All checks passed! |
good idea, added in ba1ba1b |
@gosuto-inzasheru temporarily added votes from last round to this branch to test ^^^ also updated to use the new max_omni label |
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.
great! test run in pr gives a lot more confidence. lfg!
Very nice indeed! Great work @jalbrekt85 ! |
#1517 - review_vote.py now simulates actions of vote.py, mainly parsing the vote csv and generating the EIP712 payload/message hash