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

fix format issue #6

Merged

Conversation

colinhoglund
Copy link
Contributor

This fixes a formatting issue that can be problematic when running tests in CI pipelines.

$ terraform fmt --diff --check
main.tf
diff a/main.tf b/main.tf
--- /var/folders/20/59pbfvbd0q52q8kv8j6hwdmr0000gn/T/248116428	2019-05-30 13:26:49.000000000 -0400
+++ /var/folders/20/59pbfvbd0q52q8kv8j6hwdmr0000gn/T/754506683	2019-05-30 13:26:49.000000000 -0400
@@ -31,7 +31,7 @@
   ttl     = 60

   records = [
-    "${lookup(aws_acm_certificate.this.domain_validation_options[count.index], "resource_record_value")}"
+    "${lookup(aws_acm_certificate.this.domain_validation_options[count.index], "resource_record_value")}",
   ]
 }

@antonbabenko
Copy link
Member

Thanks. Just curious which version of Terraform you are running?

@colinhoglund
Copy link
Contributor Author

Thanks. Just curious which version of Terraform you are running?

v0.11.14

@antonbabenko antonbabenko merged commit 5f83cfe into terraform-aws-modules:master May 30, 2019
@antonbabenko
Copy link
Member

v1.3.0 has been released.

@colinhoglund
Copy link
Contributor Author

v1.3.0 has been released.

Amazing! Thanks for the quick response!

@github-actions
Copy link

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 issues. 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 Nov 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants