Terraform v0.12x changes not working if TMPDIR not set with trailing slash

[spongebrain]

Thanks for the update...however, I received the following error when running:

Check for merge conflicts................................................Passed
Terraform fmt............................................................Passed
Terraform docs...........................................................Failed
hookid: terraform_docs

mktemp: failed to create file via template ‘/tmpterraform-docs-XXXXXXXXXX’: Permission denied

Looked at code and ran export TMPDIR=/tmp/, then:

$ pre-commit run -a
Check for merge conflicts................................................Passed
Terraform fmt............................................................Passed
Terraform docs...........................................................Failed
hookid: terraform_docs

2019/06/18 11:15:40 At 34:15: nested object expected: LBRACE got: ASSIGN

Thanks!

[BastianM3]

@spongebrain just ran into the same issue on 0.12.

I think it might be because of the new nested block syntax. Instead of:

something {
 property = 1
}

Something like this is wanted by Terraform:

something = {
 property = 1
}

[antonbabenko]

@spongebrain mktemp is magic. :( Please use v1.16.0 where this was fixed (see c9ecd72).

[spongebrain]

The TMPDIR issue is resolved at v1.16.0. Thanks much!!!

Insofar as the LBRACE vs ASSIGN issue...we're adopting the new v12 HCL format, and thus don't want to revert the code back to 11 syntax. Are there plans to address that in a future version?

[antonbabenko]

I am not sure that LBRACE vs ASSIGN issue you are talking about has anything to do with terraform-docs.

terraform-docs requires a code to look like Terraform 0.11 to be able to process it. This is what awk script is doing in a temporary folder.

You should be able to use Terraform 0.12 and let the terraform_docs hook do the magic.