Enabling multiplatform support by moving to python about pre-commit-terraform HOT 6 CLOSED

Very cool! Thanks for the links to those references.

from pre-commit-terraform.

I agree, we won't prioritize support for multiplatform using Python. Bash or docker is better option.

from pre-commit-terraform.

I started working on that and then found that someone had already written a script to transform the docs, which ought to be able to replace a large chunk of terraform_docs.sh. See marcuscaisey’s comment in terraform-docs/terraform-docs#62.

from pre-commit-terraform.

Not at all. I’m going to see if I can work on this and make it happen, since there’s more than just the conversion that’s written in Bash.

from pre-commit-terraform.

terraform-docs/terraform-docs#113 is now moving forward, so I’ll wait for that to land first.

from pre-commit-terraform.

Python works extremely slowly than bash. Some years ago I wrote my own hooks in python 3 and then rewrite it to sh. Same logic - x10 times faster

If you do not use WSL in Windows, the better will be dockerize hooks, than rewrite it to python. In different systems installed different python versions - this will create many issues in the future.

from pre-commit-terraform.