terraform_docs hook requires GNU sed about pre-commit-terraform HOT 4 CLOSED

First, thanks for opening this issue!

The question which I have been asking myself few times in several projects which version(s) of sed should I use. If I understand correctly GNU sed is more powerful and easily available for the most of operating systems (right?).

Will it be fine, if we just extend the script to print a statement that GNU sed should be installed? "pass" if GNU sed does not exist.

It may be possible to rewrite it using Python (or anything else), but there will be the similar issue - which version of Python to expect (python 2.7 vs 3.x?)

from pre-commit-terraform.

I ended up rewriting the sed commands in Perl, which should make the script more universally usable without external tools. See #15.

from pre-commit-terraform.

Well, not excatly because it breaks setups in small docker images where perl is not installed (like the official hashicorp/terraform:0.13.5 docker image i.e.):

Terraform docs...........................................................Failed
- hook id: terraform_docs
- exit code: 127

/builds/project-0/.cache/pre-commit/repowvx2neb8/terraform_docs.sh: line 144: perl: command not found

from pre-commit-terraform.

Make sure to install perl. I know it sounds odd in 2020 but we use it here also :)

from pre-commit-terraform.