Merge pull request #128 from trussworks/rule_set_name_required

Mark ses_rule_set as required
trussworks · Nov 16, 2022 · 9a17895 · 9a17895
2 parents ebd2755 + c1fc2af
commit 9a17895
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/README.md b/README.md
@@ -112,7 +112,7 @@ No modules.
 | receive\_s3\_kms\_key\_arn | The ARN of the KMS key for S3 objects of received emails (effective if enable\_incoming\_email is true). | `string` | `null` | no |
 | receive\_s3\_prefix | The key prefix of the S3 bucket to store received emails (required if enable\_incoming\_email is true). | `string` | `""` | no |
 | route53\_zone\_id | Route53 host zone ID to enable SES. | `string` | n/a | yes |
-| ses\_rule\_set | Name of the SES rule set to associate rules with. | `string` | `null` | no |
+| ses\_rule\_set | Name of the SES rule set to associate rules with. | `string` | n/a | yes |
 
 ## Outputs
 

diff --git a/variables.tf b/variables.tf
@@ -57,7 +57,6 @@ variable "route53_zone_id" {
 variable "ses_rule_set" {
   description = "Name of the SES rule set to associate rules with."
   type        = string
-  default     = null
 }
 
 variable "enable_incoming_email" {