-
Notifications
You must be signed in to change notification settings - Fork 10
docs: add more clear and description comments #53
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
docs: add more clear and description comments #53
Conversation
|
@hallazzang I added/updated comments in both |
…51-refactor-comments * commit '3bc1decf7cc852747e275db20404446c69d4ede9': chore: bump Cosmos SDK version to 0.44.2 test: update simulation tests (tendermint#54)
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 It seems like proto-format proto-lint proto-gen was not executed well.
This can happen if the proto docker container for another module collides even if the make is executed. let me commit this.
and Swagger was updated through this PR. The API specification is not changed but description, etc., so it is not necessary to upgrade the swagger version, but if there is a change from next time, it would be good to upgrade the swagger version from client/docs/config.json, re-generate it, and update it with the swagger hub.
Description
closes: #51
Tasks
clientfolder to improve commentskeeperfolder to improve commentsscan through(work will be done in test: randomize simulations for edge cases #32)simulationfolder to improve commentstypesfolder to improve commentsBefore we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/) or specification (x/<module>/spec/)godoccomments.Files changedin the Github PR explorerCodecov Reportin the comment section below once CI passes