Fix: Add Terraform 0.12 support to bucket_force_destroy boolean variable (#24)

* change from string to bool

* generate docs and make a test

Author: bkamin29

README.md

@@ -38,7 +38,7 @@ Then perform the following commands on the root folder:
 
 | Name | Description | Type | Default | Required |
 |------|-------------|:----:|:-----:|:-----:|
-| bucket\_force\_destroy | When deleting the GCS bucket containing the cloud function, delete all objects in the bucket first. | string | `"true"` | no |
+| bucket\_force\_destroy | When deleting the GCS bucket containing the cloud function, delete all objects in the bucket first. | bool | `"true"` | no |
 | bucket\_name | The name to apply to the bucket. Will default to a string of <project-id>-scheduled-function-XXXX> with XXXX being random characters. | string | `""` | no |
 | function\_available\_memory\_mb | The amount of memory in megabytes allotted for the function to use. | number | `"256"` | no |
 | function\_description | The description of the function. | string | `"Processes log export events provided through a Pub/Sub topic subscription."` | no |

variables.tf

@@ -117,8 +117,8 @@ variable "bucket_name" {
 }
 
 variable "bucket_force_destroy" {
-  type        = "string"
-  default     = "true"
+  type        = bool
+  default     = true
   description = "When deleting the GCS bucket containing the cloud function, delete all objects in the bucket first."
 }