Return first non-empty YAML document when parsing YAML streams #683
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This relates to #534, and while full multi-document stream support is going to take a bit longer to fully figure out, I think this is a quick win.
Basically this handles the (not uncomment in Helm and Kubernetes-land) case where a file like:
will get parsed by gomplate as just a single
nil
, which is pretty much never what we actually want.So now YAML will ignore non-empty documents and return the first one in the stream.
Signed-off-by: Dave Henderson dhenderson@gmail.com