-
Notifications
You must be signed in to change notification settings - Fork 1
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: E2E challenge test and loads of bug fixes #32
Conversation
…iask--fix-length-trust-assumption
…needs better translation for bytecode --> abi types
…rator proof generation tests
} | ||
}; | ||
} | ||
|
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.
nit: might be a better way to do this but i guess its fine for now.
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.
yeah unless we pass it through via cfg ingestion which would be pretty painful since it'd require instrumentation of the wavm machine which would require exposing rust functionality --> C --> Go
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.
generally lgtm, approving to get things rolling
msgData := []byte{daprovider.EigenDAMessageHeaderFlag} | ||
msgData = append(msgData, b...) | ||
|
||
return msgData, nil |
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.
probably should add a comment as to why the above is needed and done the way it is.
Dependency PRs: