Skip to content

Commit

Permalink
Specify all variables in the basic_usage example as non-nullable
Browse files Browse the repository at this point in the history
In #221 we made all of the variables in the example module non-nullable
but did not do the same to the `basic_usage` example. Since it uses
variables beyond the requirements of the example module it makes sense
to also specify the example's variables as non-nullable.
  • Loading branch information
mcdonnnj committed Nov 13, 2024
1 parent e7315c0 commit 0a0553a
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions examples/basic_usage/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@

variable "tf_role_arn" {
description = "The ARN of the role that can terraform non-specialized resources."
nullable = false
type = string
}

Expand All @@ -18,18 +19,21 @@ variable "tf_role_arn" {
variable "ami_owner_account_id" {
default = "self"
description = "The ID of the AWS account that owns the AMI, or \"self\" if the AMI is owned by the same account as the provisioner."
nullable = false
type = string
}

variable "aws_availability_zone" {
default = "a"
description = "The AWS availability zone to deploy into (e.g. a, b, c, etc.)."
nullable = false
type = string
}

variable "aws_region" {
default = "us-east-1"
description = "The AWS region to deploy into (e.g. us-east-1)."
nullable = false
type = string
}

Expand All @@ -38,5 +42,6 @@ variable "tags" {
Testing = true
}
description = "Tags to apply to all AWS resources created."
nullable = false
type = map(string)
}

0 comments on commit 0a0553a

Please sign in to comment.