better handle missing dependancies about pre-commit-terraform HOT 6 CLOSED

Hi Christopher,

There can be a way to verify that dependencies are installed, but it will make invocation much slower.

I agree we could make README more complete and describe dependencies more accurately. Pull request and ideas are welcome.

from pre-commit-terraform.

This issue has been automatically marked as stale because it has been open 30 days
with no activity. Remove stale label or comment or this issue will be closed in 10 days

from pre-commit-terraform.

Not sure if this was discussed in a different issue, but I see that some other hooks ship with dependencies, or download them if missing (not sure which); what if pre-commit-terraform did so with it's dependencies?

The specific problem we're running into is that our pre-commit.ci run is failing due to Terraform not being installed in the runner.

Note this would also make installation easier for those that might not have Terraform installed (#445), such as a monorepo where Terraform is mixed with app code, worked on by different people.

Thanks!

from pre-commit-terraform.

@afeld that's not possible to do natively via pre-commit, check pre-commit/pre-commit#1453

from pre-commit-terraform.

I forked the shellcheck-py approach and did the exact same for terraform: https://github.com/yuvipanda/terraform-bin. It's not published on PyPI, but this approach works fine for pre-commit.ci!

I currently have a pre-commit-hooks.yaml file setup on that repo so it is used directly. However, if possible, perhaps it could be made a dependency of this repo, so I could use this repo instead? This way, it would work fine on pre-commit.ci without any issues as well :) Am happy to transfer ownership, etc as needed too.

from pre-commit-terraform.

@yuvipanda if you'd like to talk about this - please open a new issue.

You're the first folk on my way who use pre-commit.ci, when they can be run in the company's CIs or as GitHub workflow. Also, 100 users are a quite low limit.

So, I don't think that supporting a solution that will work only with this CI is worth anything.

If it can run not only in pre-commit.ci, then it relates to #418 a little bit.

from pre-commit-terraform.