Address PR comments

Author: mbarrien

aws-acm-certificate/outputs.tf

@@ -1,7 +1,9 @@
 output "arn" {
-  value = aws_acm_certificate.cert.arn
+  value       = aws_acm_certificate.cert.arn
+  description = "AWS ARN of the certificate"
 }
 
 output "id" {
-  value = aws_acm_certificate.cert.id
+  value       = aws_acm_certificate.cert.id
+  description = "ID of the certificate"
 }

aws-acm-certificate/variables.tf

@@ -10,16 +10,17 @@ variable "cert_subject_alternative_names" {
 }
 
 variable "aws_route53_zone_id" {
-  type = string
+  type        = string
+  description = "Default Route 53 zone to create validation records in"
 }
 
 variable "validation_record_ttl" {
-  type    = string
-  default = 60
+  type        = string
+  default     = 60
+  description = "TTL value of DNS validation records"
 }
 
 variable tags {
-  type        = map(string)
+  type        = object({ project : string, env : string, service : string, owner : string, managedBy : string })
   description = "Tags to apply to certificate"
-  default     = {}
 }