-
Notifications
You must be signed in to change notification settings - Fork 4
feat: Add support to display mandatory and optional flags separately in CLI help #62
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
Conversation
|
updated cli commands: base64-tgz decrypt-attestation download-certificate encrypt encrypt-string get-certificate image validate-contract validate-encryption-certificate validate-network |
Sashwat-K
left a comment
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.
Did you run make tidy ?
Sashwat-K
left a comment
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. Thank you @vikas-sharma24 for the continued contributions!
|
🎉 This PR is included in version 1.12.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Add support to display mandatory and optional flags separately in CLI help
Sample CLI help command output:
Description
CLI currently does not differentiate between mandatory and optional flags in the help output. All flags are listed together, and users must rely on error messages to discover which flags are required. So enhancing the CLI to display required and optional flags separately.
Related Issue
Fixes #61
Type of Change
Testing
Describe the tests you ran and how to reproduce them
Checklist