Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

helper/structure: Handle nil interface #666

Merged
merged 1 commit into from
Dec 3, 2018
Merged

Conversation

bill-rich
Copy link
Contributor

Ignore nil interfaces when converting a slice of interfaces into a slice
of strings. This becomes an issue when a list with an empty string is
passed to SliceInterfacesToStrings.

Fixes #643

Ignore nil interfaces when converting a slice of interfaces into a slice
of strings.
@bill-rich bill-rich requested a review from a team December 3, 2018 17:53
@ghost ghost added the size/xs Relative Sizing: Extra-Small label Dec 3, 2018
Copy link

@tombuildsstuff tombuildsstuff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@bill-rich bill-rich merged commit 1bdcdb6 into master Dec 3, 2018
@bill-rich bill-rich deleted the b-handle-nil-interface branch December 3, 2018 18:18
@ghost ghost locked and limited conversation to collaborators Apr 18, 2020
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
size/xs Relative Sizing: Extra-Small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Terraform crash when applying vmware config
2 participants