-
-
Notifications
You must be signed in to change notification settings - Fork 597
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
feat(cli-executer): validate user input #1610
feat(cli-executer): validate user input #1610
Conversation
a72f0d9
to
19e823d
Compare
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.
Can we test it something?
We already have the unit tests for |
29a4fc9
19e823d
to
29a4fc9
Compare
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.
Feel free to merge
@jamesgeorge007 Thanks for your update. I labeled the Pull Request so reviewers will review it again. @rishabh3112 Please review the new changes. |
What kind of change does this PR introduce?
enhancement
Did you add tests for your changes?
Nope
If relevant, did you update the documentation?
Nope
Summary
An in-house warning shows up if the user doesn't supply a value for required flags as part of
cli-executer
.Does this PR introduce a breaking change?
Nope
Other information
N/A