Skip to content

go get github.com/zclconf/go-cty-yaml@v1.0.3#32151

Merged
apparentlymart merged 1 commit intomainfrom
b-yamldecode-emptydoc-null
Nov 30, 2022
Merged

go get github.com/zclconf/go-cty-yaml@v1.0.3#32151
apparentlymart merged 1 commit intomainfrom
b-yamldecode-emptydoc-null

Conversation

@apparentlymart
Copy link
Copy Markdown
Contributor

This includes the fix for a bug in what Terraform calls the "yamldecode" function, where it was not correctly handling any situation where the decode result is a null value. It was previously returning an unknown value in that case, whereas now it returns a null value as expected.

Fixes #31188.

This includes the fix for a bug in what Terraform calls the "yamldecode"
function, where it was not correctly handling any situation where the
decode result is a null value. It was previously returning an unknown
value in that case, whereas now it returns a null value as expected.
@apparentlymart apparentlymart force-pushed the b-yamldecode-emptydoc-null branch from a0b57d0 to f05e913 Compare November 30, 2022 01:27
@apparentlymart apparentlymart merged commit 8253821 into main Nov 30, 2022
@apparentlymart apparentlymart deleted the b-yamldecode-emptydoc-null branch November 30, 2022 01:45
@github-actions
Copy link
Copy Markdown
Contributor

Reminder for the merging maintainer: if this is a user-visible change, please update the changelog on the appropriate release branch.

@github-actions
Copy link
Copy Markdown
Contributor

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unknown values when decoding empty but yaml documents

2 participants