Fix: Inconsistent conditional result types (cloudposse#68)

* use for filter directly rather than ternary

* remove unused provider and update minimum terraform version

* Auto Format

* revert version bump

* Auto Format

Co-authored-by: cloudpossebot <[email protected]>

Author: a296Cg

README.md

@@ -287,7 +287,6 @@ Available targets:
 | Name | Version |
 |------|---------|
 | <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 0.13.0 |
-| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 3.0 |
 | <a name="requirement_datadog"></a> [datadog](#requirement\_datadog) | >= 3.0.0 |
 
 ## Providers

docs/terraform.md

@@ -4,7 +4,6 @@
 | Name | Version |
 |------|---------|
 | <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 0.13.0 |
-| <a name="requirement_aws"></a> [aws](#requirement\_aws) | >= 3.0 |
 | <a name="requirement_datadog"></a> [datadog](#requirement\_datadog) | >= 3.0.0 |
 
 ## Providers

modules/monitors/main.tf

@@ -6,7 +6,7 @@ locals {
 
 # https://registry.terraform.io/providers/DataDog/datadog/latest/docs/resources/monitor
 resource "datadog_monitor" "default" {
-  for_each = local.enabled ? { for k, v in var.datadog_monitors : k => v if lookup(v, "enabled", true) } : {}
+  for_each = { for k, v in var.datadog_monitors : k => v if local.enabled && lookup(v, "enabled", true) }
 
   name                   = each.value.name
   type                   = each.value.type

modules/monitors/versions.tf

@@ -2,10 +2,6 @@ terraform {
   required_version = ">= 0.13.0"
 
   required_providers {
-    aws = {
-      source  = "hashicorp/aws"
-      version = ">= 3.0"
-    }
     datadog = {
       source  = "datadog/datadog"
       version = ">= 3.0.0"

versions.tf

@@ -2,10 +2,6 @@ terraform {
   required_version = ">= 0.13.0"
 
   required_providers {
-    aws = {
-      source  = "hashicorp/aws"
-      version = ">= 3.0"
-    }
     datadog = {
       source  = "datadog/datadog"
       version = ">= 3.0.0"