-
Notifications
You must be signed in to change notification settings - Fork 92
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
eth/client: AuditContract #1319
Conversation
0a7c717
to
7beee34
Compare
7beee34
to
7df7fc2
Compare
7df7fc2
to
d0f7da8
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.
I'm embarrassingly not up to date with all the communication changes, but this looks good to me.
b6b638d
to
03be944
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.
I was holding off on reviewing this until the ParseInitiateData
changes come through, but if it's important to anyone to get it earlier, we can.
I dunno if order matters too much. Whoever does the ParseInitiateData etc changes can apply it to AuditContract too. |
I'm working on them right now, but yeah this can be merged first. |
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.
This looks good for now. Obviously a lot will be changing here with versioned ParseInitiateData
and #1320.
03be944
to
39b1005
Compare
39b1005
to
092e5ab
Compare
Part of #1154