Skip to content
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

Always use json format for inspec report #212

Merged
merged 1 commit into from
May 3, 2017

Conversation

chris-rock
Copy link
Contributor

@chris-rock chris-rock commented Apr 26, 2017

Description

This will run inspec just once, no matter if the report needs to be done in full json or min json.

Issues Resolved

Check List

Signed-off-by: Christoph Hartmann <chris@lollyrock.com>
@chris-rock chris-rock force-pushed the chris-rock/improve-reporting branch from 6d9fb51 to edbd9c9 Compare May 3, 2017 09:00
@chris-rock chris-rock changed the title WIP: always use json reporter always use json reporter May 3, 2017
@chris-rock chris-rock changed the title always use json reporter always use json format for inspec May 3, 2017
@chris-rock chris-rock changed the title always use json format for inspec WIP: always use json format for inspec May 3, 2017
@chris-rock chris-rock changed the title WIP: always use json format for inspec Always use json format for inspec report May 3, 2017
Copy link
Contributor

@arlimus arlimus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great improvement, kudos @chris-rock !

@arlimus arlimus merged commit 839addb into master May 3, 2017
@arlimus arlimus deleted the chris-rock/improve-reporting branch May 3, 2017 10:50
Copy link

@krsh-off krsh-off left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello,

Sorry if I'm confused, but how this condition may pass if we explicitly set format to json (in line 51) ?
Just trying to find a way to print Inspec summary in Chef logs and don't see it's possible now :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants