-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
update review function names #1657
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.
Just one question:
Sometimes we use "Despliegue de contrato" (Uppercase on the first word and the rest lower case) and other times we use "Contrato Desplegado" or "Interacción con Contrato" (Uppercases after the first word). Is this intentional?
Otherwise, LGTM! 👍
Also looks like there is still some discussion in regards to the actual naming convention to be used |
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.
Looks good on both OS's, QA Passed 👍
* update * snapshot * :unknown:
Description
Contract Interaction
instead ofUnknown Method
Checklist
Issue
Resolves #???