diff --git a/modules/gcs/README.md b/modules/gcs/README.md
index fdba89e60..627ce93a2 100644
--- a/modules/gcs/README.md
+++ b/modules/gcs/README.md
@@ -97,9 +97,9 @@ module "buckets" {
 | *iam_roles* | IAM roles keyed by bucket name. | <code title="map&#40;list&#40;string&#41;&#41;">map(list(string))</code> |  | <code title="">{}</code> |
 | *labels* | Labels to be attached to all buckets. | <code title="map&#40;string&#41;">map(string)</code> |  | <code title="">{}</code> |
 | *location* | Bucket location. | <code title="">string</code> |  | <code title="">EU</code> |
-| *logging* | Per-bucket logging. | <code title="map&#40;map&#40;string&#41;&#41;">map(object)</code> |  | <code title="">{}</code> |
+| *logging_config* | Per-bucket logging. | <code title="map&#40;object&#40;&#123;&#10;log_bucket        &#61; string&#10;log_object_prefix &#61; string&#10;&#125;&#41;&#41;">map(object({...}))</code> |  | <code title="">{}</code> |
 | *prefix* | Prefix used to generate the bucket name. | <code title="">string</code> |  | <code title="">null</code> |
-| *retention_policies* | Per-bucket retention policy. | <code title="map&#40;map&#40;string&#41;&#41;">map(object)</code> |  | <code title="">{}</code> |
+| *retention_policies* | Per-bucket retention policy. | <code title="map&#40;object&#40;&#123;&#10;retention_period &#61; number&#10;is_locked        &#61; bool&#10;&#125;&#41;&#41;">map(object({...}))</code> |  | <code title="">{}</code> |
 | *storage_class* | Bucket storage class. | <code title="">string</code> |  | <code title="">MULTI_REGIONAL</code> |
 | *versioning* | Optional map to set versioning keyed by name, defaults to false. | <code title="map&#40;bool&#41;">map(bool)</code> |  | <code title="">{}</code> |
 
diff --git a/modules/gcs/main.tf b/modules/gcs/main.tf
index 44feda5fb..92a84107d 100644
--- a/modules/gcs/main.tf
+++ b/modules/gcs/main.tf
@@ -36,9 +36,15 @@ locals {
     ? ""
     : join("-", [var.prefix, lower(var.location), ""])
   )
-  kms_keys = { for name in var.names : name => lookup(var.encryption_keys, name, null) }
-  retention_policy = { for name in var.names : name => lookup(var.retention_policies, name, null) }
-  logging_config = { for name in var.names : name => lookup(var.logging_config, name, null) }
+  kms_keys = {
+    for name in var.names : name => lookup(var.encryption_keys, name, null)
+  }
+  retention_policy = {
+    for name in var.names : name => lookup(var.retention_policies, name, null)
+  }
+  logging_config = {
+    for name in var.names : name => lookup(var.logging_config, name, null)
+  }
 }
 
 resource "google_storage_bucket" "buckets" {
@@ -70,14 +76,14 @@ resource "google_storage_bucket" "buckets" {
     for_each = local.retention_policy[each.key] == null ? [] : [""]
     content {
       retention_period = local.retention_policy[each.key]["retention_period"]
-      is_locked = local.retention_policy[each.key]["is_locked"]
+      is_locked        = local.retention_policy[each.key]["is_locked"]
     }
   }
 
   dynamic logging {
     for_each = local.logging_config[each.key] == null ? [] : [""]
     content {
-      log_bucket = local.logging_config[each.key]["log_bucket"]
+      log_bucket        = local.logging_config[each.key]["log_bucket"]
       log_object_prefix = local.logging_config[each.key]["log_object_prefix"]
     }
   }
diff --git a/modules/gcs/variables.tf b/modules/gcs/variables.tf
index 790381659..5ea231b20 100644
--- a/modules/gcs/variables.tf
+++ b/modules/gcs/variables.tf
@@ -87,17 +87,17 @@ variable "versioning" {
 variable "retention_policies" {
   description = "Per-bucket retention policy."
   type = map(object({
-  		retention_period = number
-		is_locked        = bool
-	}))
-  default     = {}
+    retention_period = number
+    is_locked        = bool
+  }))
+  default = {}
 }
 
 variable "logging_config" {
   description = "Per-bucket logging."
   type = map(object({
-  		log_bucket = string
-  		log_object_prefix = string
-	}))
-  default     = {}
+    log_bucket        = string
+    log_object_prefix = string
+  }))
+  default = {}
 }