Skip to content
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 a download file name and add test cases #1

Merged

Conversation

tnasu
Copy link

@tnasu tnasu commented Oct 7, 2022

Description

  • Update download file name
  • Update the query command description
  • Add test cases

Motivation and context

How has this been tested?

Screenshots (if appropriate):

Checklist:

  • I followed the contributing guidelines and code of conduct.
  • I have added a relevant changelog to CHANGELOG.md
  • I have added tests to cover my changes.
  • I have updated the documentation accordingly.
  • I have updated API documentation client/docs/swagger-ui/swagger.yaml

@da1suk8 da1suk8 merged commit 1f70048 into da1suk8:fix/fix_GetCmdQueryCode Oct 11, 2022
da1suk8 added a commit that referenced this pull request Oct 12, 2022
* fix: fix to prevent accepting file name

* Update a download file name and add test cases (#1)

* Update short/long usage and change filename

* Add query_test

* Add test cases

* fix: go.mod

* docs: update CHANGELOG.md

Co-authored-by: Toshimasa Nasu <toshimasa_nasu@hotmail.com>
@tnasu tnasu deleted the fix/fix_GetCmdQueryCode branch October 17, 2022 05:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants