Merge pull request #15 from trussworks/resolve-master-error

update terraform format and docs

README.md

@@ -100,27 +100,6 @@ No modules.
 ## Inputs
 
 | Name | Description | Type | Default | Required |
-|------|-------------|------|---------|:-----:|
-| alarm\_namespace | Namespace for generated Cloudwatch alarms | `string` | `"CISBenchmark"` | no |
-| alarm\_sns\_topic\_arn | SNS topic ARN for generated alarms | `string` | n/a | yes |
-| aws\_config\_changes | Toggle AWS Config changes alarm | `bool` | `true` | no |
-| cloudtrail\_cfg\_changes | Toggle Cloudtrail config changes alarm | `bool` | `true` | no |
-| cloudtrail\_log\_group\_name | Cloudwatch log group name for Cloudtrail logs | `string` | `"cloudtrail-events"` | no |
-| console\_signin\_failures | Toggle console signin failures alarm | `bool` | `true` | no |
-| disable\_assumed\_role\_login\_alerts | Toggle to disable assumed role console login alerts - violates CIS Benchmark | `bool` | `false` | no |
-| disable\_or\_delete\_cmk | Toggle disable or delete CMK alarm | `bool` | `true` | no |
-| iam\_changes | Toggle IAM changes alarm | `bool` | `true` | no |
-| nacl\_changes | Toggle network ACL changes alarm | `bool` | `true` | no |
-| network\_gw\_changes | Toggle network gateway changes alarm | `bool` | `true` | no |
-| no\_mfa\_console\_login | Toggle no MFA console login alarm | `bool` | `true` | no |
-| root\_usage | Toggle root usage alarm | `bool` | `true` | no |
-| route\_table\_changes | Toggle route table changes alarm | `bool` | `true` | no |
-| s3\_bucket\_policy\_changes | Toggle S3 bucket policy changes alarm | `bool` | `true` | no |
-| security\_group\_changes | Toggle security group changes alarm | `bool` | `true` | no |
-| tags | Tags for resources created | `map(string)` | `{}` | no |
-| unauthorized\_api\_calls | Toggle unauthorized api calls alarm | `bool` | `true` | no |
-| vpc\_changes | Toggle VPC changes alarm | `bool` | `true` | no |
-=======
 |------|-------------|------|---------|:--------:|
 | <a name="input_alarm_namespace"></a> [alarm\_namespace](#input\_alarm\_namespace) | Namespace for generated Cloudwatch alarms | `string` | `"CISBenchmark"` | no |
 | <a name="input_alarm_sns_topic_arn"></a> [alarm\_sns\_topic\_arn](#input\_alarm\_sns\_topic\_arn) | SNS topic ARN for generated alarms | `string` | n/a | yes |
@@ -138,6 +117,7 @@ No modules.
 | <a name="input_route_table_changes"></a> [route\_table\_changes](#input\_route\_table\_changes) | Toggle route table changes alarm | `bool` | `true` | no |
 | <a name="input_s3_bucket_policy_changes"></a> [s3\_bucket\_policy\_changes](#input\_s3\_bucket\_policy\_changes) | Toggle S3 bucket policy changes alarm | `bool` | `true` | no |
 | <a name="input_security_group_changes"></a> [security\_group\_changes](#input\_security\_group\_changes) | Toggle security group changes alarm | `bool` | `true` | no |
+| <a name="input_tags"></a> [tags](#input\_tags) | Tags for resources created | `map(string)` | `{}` | no |
 | <a name="input_unauthorized_api_calls"></a> [unauthorized\_api\_calls](#input\_unauthorized\_api\_calls) | Toggle unauthorized api calls alarm | `bool` | `true` | no |
 | <a name="input_vpc_changes"></a> [vpc\_changes](#input\_vpc\_changes) | Toggle VPC changes alarm | `bool` | `true` | no |
 

main.tf

@@ -1,5 +1,5 @@
 locals {
- resource_tags = merge(var.tags, {"Automation"="Terraform"})
+ resource_tags = merge(var.tags, { "Automation" = "Terraform" })
 }
 
 resource "aws_cloudwatch_log_metric_filter" "unauthorized_api_calls" {