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
Document additional usage of json resource #3737
Conversation
The `command` and `content` usage seems to have been supported for some time, but is not included in the documentation. ref: * inspec#1271 Signed-off-by: Sean Escriva <sean.escriva@gmail.com>
Force pushed to update the signoff for DCO, let me know if I missed anything here |
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.
Thank you for this @webframp! Just a few typos then we'll get it merged.
Also, congratulations on your first contribution. 😄
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.
Whoops, missed one on the other review.
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.
Thanks @webframp! I think this is safe to merge after we fix up the typos.
Signed-off-by: Sean Escriva <sean.escriva@gmail.com>
Thanks for the review @miah and @jerryaldrichiii and @Vasu1105. Updated my branch |
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.
Awesome work @webframp.
The
command
andcontent
usage seems to have been supported for some time, but is not included in the documentation.ref: