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

verify yaml2json before using it #46

Merged
merged 1 commit into from
Nov 4, 2017
Merged

verify yaml2json before using it #46

merged 1 commit into from
Nov 4, 2017

Conversation

aeruder
Copy link
Contributor

@aeruder aeruder commented Oct 31, 2017

This addresses #44 which had an issue when yaml2json in the PATH was not
something as simple as reads from stdin/writes to stdout. In order to
test, let's just put together a really simple yaml file consisting of:

---
test: 1

and make sure we can parse it and json_decode it.

This addresses rafi#44 which had an issue when yaml2json in the PATH was not
something as simple as reads from stdin/writes to stdout.  In order to
test, let's just put together a really simple yaml file consisting of:

    ---
    test: 1

and make sure we can parse it and json_decode it.
@rafi rafi merged commit ac4fd8c into rafi:master Nov 4, 2017
@rafi
Copy link
Owner

rafi commented Nov 4, 2017

Appreciate it, thanks!

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.

None yet

2 participants