hook fails if comment tags not found about pre-commit-terraform HOT 3 CLOSED

My reasoning for making it using comments was that not everybody wants to overwrite the whole README.md but rather put it into specified section of that file.

Later we may have a possibility to specify which file to update instead of README.md as now.

Regarding the issue with sed... unexpected EOF. It is happening not because a comment tag was not found, but because sed command does not work on all platforms equally well (I tested it with brew'ed sed on Mac). The most platform agnostic way seems to be using Perl (#15).

I will keep this issue posted when a new pre-commit hook is released (during today or tomorrow).

from pre-commit-terraform.

v1.7.2 is using Perl and keeps a temporary file in a temporary directory.

Please give it a try and let us know how it goes.

from pre-commit-terraform.

v1.7.3 at least got me past this error. I wanted all of the documentation in main.tf to overwrite the README.md. The hook doesn't actually write the README contents like #7 asks for still, but this could probably be closed.

from pre-commit-terraform.