Skip to content

Conversation

@faulkner
Copy link

Type is optional (https://www.terraform.io/docs/configuration/variables.html#type), so infer the type when possible.

This fixes panics for cases like:

variable "foo" {
  default = []
}

Type is optional (https://www.terraform.io/docs/configuration/variables.html#type), so infer the type when possible.

This fixes panics for cases like:

```
variable "foo" {
  default = []
}
```
@metmajer
Copy link
Member

Great stuff @faulkner, thanks for the contribution!

@metmajer metmajer merged commit 6411d80 into getcloudnative:master Sep 11, 2018
@metmajer
Copy link
Member

@faulkner I wanted to give you heads up that that maintenance is continued at segmentio/terraform-docs!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants