-
Notifications
You must be signed in to change notification settings - Fork 93
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
Refactor OperatorMsg
#3773
Refactor OperatorMsg
#3773
Conversation
Signed-off-by: Jeremiah Corrado <[email protected]>
Signed-off-by: Jeremiah Corrado <[email protected]>
Signed-off-by: Jeremiah Corrado <[email protected]>
OperatroMsg
Signed-off-by: Jeremiah Corrado <[email protected]>
Signed-off-by: Jeremiah Corrado <[email protected]>
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 doing this! It's so much easier to read now.
The |
Signed-off-by: Jeremiah Corrado <[email protected]>
I think that's a good reason to leave it. If you feed so inclined, you could make them like: // " <- quote is workaround for syntax highlighter bug but I don't think it's super necessary |
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 so much jeremiah!! im sure this was terribly tedious
Co-authored-by: tess <[email protected]>
Co-authored-by: tess <[email protected]>
Co-authored-by: tess <[email protected]>
Refactor
operatorMsg
to useinstantiateAndRegister
.Because binary operations have traditionally not supported
uint(8)
, a separate parameter-class is added with all the scalar types exceptuint(8)
.