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

Add support for platform and suite flag to reporter configuration option #198

Closed
wants to merge 1 commit into from

Conversation

wiebe
Copy link
Contributor

@wiebe wiebe commented Aug 22, 2018

This makes it possible to create custom file paths more easily

This makes it possible to create custom file paths more easily

Signed-off-by: Wiebe Verweij <wiebe@wiebelt.nl>
@b-dean
Copy link

b-dean commented Aug 22, 2018

Not sure if you should do it, but someone should update the part of the README.md that talks about using junit output.

@jquick
Copy link
Contributor

jquick commented Sep 6, 2018

Hey @wiebe ! Thanks for this addition. I think this other PR covered this here #200. Thanks for getting attention to this!

@jquick jquick closed this Sep 6, 2018
@wiebe
Copy link
Contributor Author

wiebe commented Sep 6, 2018

You can specify multiple reporters to inspec, i'm not sure the other pull request/version works likes my version. Unless ruby automagically substitutes it on every item in the array? (that is why i used .map)

Or you have to specify the multiple reporters as one string in the yaml file I guess? But that seems less readable to me.

I'll test it tomorrow if i have time.

@wiebe
Copy link
Contributor Author

wiebe commented Sep 7, 2018

It doesn't work at all, see #202

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

Successfully merging this pull request may close these issues.

3 participants