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

Remove problematic conditional #31

Merged
merged 1 commit into from
Sep 26, 2023
Merged

Remove problematic conditional #31

merged 1 commit into from
Sep 26, 2023

Conversation

Nuru
Copy link
Contributor

@Nuru Nuru commented Sep 26, 2023

what

  • Remove problematic conditional

why

  • local.deprecated_statements_values can be a tuple, and Terraform does not have a concept of an empty or null tuple to use as an alternative in a conditional, so you can get an error like:
The true and false result expressions must have consistent types. The 'true' tuple has length 0, but the 'false' tuple has length 2.

references

@Nuru Nuru added the bugfix Change that restores intended behavior label Sep 26, 2023
@Nuru Nuru requested review from a team as code owners September 26, 2023 02:19
@Nuru
Copy link
Contributor Author

Nuru commented Sep 26, 2023

/terratest

@Nuru Nuru enabled auto-merge (squash) September 26, 2023 02:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Change that restores intended behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants