-
Notifications
You must be signed in to change notification settings - Fork 683
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
Fix command evaluation for inspec shell -c #943
Conversation
f97d50c
to
7f2c906
Compare
7f2c906
to
c806502
Compare
8c9bba5
to
d3b6875
Compare
d3b6875
to
8d82015
Compare
Fixed #942 |
@stevendanna @chris-rock @arlimus @vjeffrey Yay tests pass. I think we have a good set of functional tests. Clearly there can be more. I kinda want to get these in before I spend more time over testing this. |
@@ -162,22 +162,23 @@ def shell_func | |||
diagnose | |||
o = opts.dup | |||
|
|||
log_device = opts['format'] == 'json' ? nil : STDOUT | |||
json_output = ['json', 'json-min'].include?(opts['format']) |
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.
Great catch!!
These look great, especially the large selection of additional tests. There can always be more, but the tricky part is to select the right amount of tests and I like the selection you've made. The only slight thing we should keep in mind is this: 👍 Thank you Kartik, this is a great improvement 😄 |
8d82015
to
039c760
Compare
No description provided.