fix: var.app_installations should be of type set(string) instead of s…

…et(number)
mineiros-io · May 10, 2022 · 5a2dad9 · 5a2dad9
1 parent 5eaaad1
commit 5a2dad9
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 6 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+
+- `var.app_installations` should be a of type `set(string)` instead of `set(number)`
+
 ## [0.16.1]
 
 ### Added

diff --git a/README.md b/README.md
@@ -811,7 +811,7 @@ This is due to some terraform limitation and we will update the module once terr
 
 #### App Installations
 
-- [**`app_installations`**](#var-app_installations): *(Optional `set(number)`)*<a name="var-app_installations"></a>
+- [**`app_installations`**](#var-app_installations): *(Optional `set(string)`)*<a name="var-app_installations"></a>
 
   A set of GitHub App IDs to be installed in this repository.
 
@@ -820,7 +820,7 @@ This is due to some terraform limitation and we will update the module once terr
   Example:
 
   ```hcl
-  app_installations = [25405144, 12556423]
+  app_installations = ["05405144", "12556423"]
   ```
 
 ### Module Configuration

diff --git a/README.tfdoc.hcl b/README.tfdoc.hcl
@@ -1057,14 +1057,14 @@ section {
         title = "App Installations"
 
         variable "app_installations" {
-          type        = set(number)
+          type        = set(string)
           default     = {}
           description = <<-END
             A set of GitHub App IDs to be installed in this repository.
           END
 
           readme_example = <<-END
-            app_installations = [25405144, 12556423]
+            app_installations = ["05405144", "12556423"]
           END
         }
       }

diff --git a/test/unit-complete/variables.tf b/test/unit-complete/variables.tf
@@ -257,7 +257,7 @@ variable "autolink_references" {
 }
 
 variable "app_installations" {
-  type        = set(number)
+  type        = set(string)
   description = "A list of GitHub App IDs to be installed in this repository."
   default     = []
 }
diff --git a/variables.tf b/variables.tf
@@ -529,7 +529,7 @@ variable "archive_on_destroy" {
 }
 
 variable "app_installations" {
-  type        = set(number)
+  type        = set(string)
   description = "(Optional) A list of GitHub App IDs to be installed in this repository."
   default     = []
 }