(Feature) Support terraform.workspace variable #181
Conversation
|
Currently this just blindly sets Ideally this would actually set it's value from the |
|
Thanks for your contribution!
I guess it will behave similarly to |
|
Ah yes, I'll add that to the PR 👍 |
* Using ${terraform.workspace} currently causes `tflint` to fail:
`unknown variable accessed: terraform.workspace in terraform.tf`
* Resolves terraform-linters#177
Stretch96
force-pushed
the
feature/support-terraform-workspace-variable
branch
from
39b20d6
to
d0d20e6
Compare
|
@wata727 That's added to the PR now 👍 |
| @@ -25,6 +25,10 @@ func NewEvaluator(templates map[string]*hclast.File, schema []*schema.Template, | |||
| Type: hilast.TypeString, | |||
| Value: c.TerraformEnv, | |||
| } | |||
| varMap["terraform.workspace"] = hilast.Variable{ | |||
There was a problem hiding this comment.
Please add a test case for ${terraform.workspace} syntax :)
Example:
https://github.com/wata727/tflint/blob/3cf4a7eff3d7869da768ae3e51d8699d77c4c1df/evaluator/evaluator_test.go#L267-L295
There was a problem hiding this comment.
Yep sure, I'll add that in soon 😬
There was a problem hiding this comment.
Ready for review 👍
Stretch96
force-pushed
the
feature/support-terraform-workspace-variable
branch
from
e4bf19c
to
2ee8232
Compare
|
👍 |
tflintto fail:unknown variable accessed: terraform.workspace in terraform.tf