-
Notifications
You must be signed in to change notification settings - Fork 247
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 opm commands to specifcy if they don't take args #786
Update opm commands to specifcy if they don't take args #786
Conversation
Codecov Report
@@ Coverage Diff @@
## master #786 +/- ##
==========================================
+ Coverage 50.65% 50.67% +0.01%
==========================================
Files 102 102
Lines 8879 8879
==========================================
+ Hits 4498 4499 +1
+ Misses 3530 3529 -1
Partials 851 851
Continue to review full report at Codecov.
|
Signed-off-by: Tyler Slaton <tyslaton@redhat.com>
01f2071
to
d1388ba
Compare
/lgtm |
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.
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: njhale, tylerslaton The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…-framework#786) Signed-off-by: Tyler Slaton <tyslaton@redhat.com> Signed-off-by: akihikokuroda <akihikokuroda2020@gmail.com>
Description of the change:
Updating the
opm
binary to specify when a command takes arguments or not. This will prevent typos from successfully running should a user accidentally type args into a command that does not accept them.Motivation for the change:
Issue 703
Reviewer Checklist
/docs
Closes 703