fix: checking of arguments - Githubissues

superfaceai / cli

Let AI connect the APIs for you

https://superface.ai

MIT License

22 stars 4 forks source link

fix: checking of arguments #341

Closed Jakub-Vacek closed 1 year ago

Jakub-Vacek commented 1 year ago

Description

Fixes: https://github.com/superfaceai/cli/issues/340 Also fixes other commands with two positional arguments where one missing argument can lead to not relevant error message

Motivation and Context

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have updated the documentation accordingly. For updating Oclif commands documentation use oclif-dev.
[ ] I have read the CONTRIBUTING document.
[ ] I haven't repeated the code. (DRY)
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Jakub-Vacek commented 1 year ago

Is it possible in OCLIF to trigger printout of the USAGE part of the command help msg?

I think there is no easy way now. I tried to add function to our Command base class but all of relevant data lives on static side of classes and is accessible after parse. So I would need to prepare some function that takes instance of Command, picks data (args, descriptions etc.) and prints them to string.