New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

allow for docker hooks #197

Closed

kirinnee opened this issue May 1, 2021 · 1 comment

Assignees

Labels

estimate/2h feature

kirinnee commented May 1, 2021

Allow for running hooks in docker (to remove dependencies), similar to the terraform docs's original hook:

/~https://github.com/terraform-docs/terraform-docs/blob/master/docs/user-guide/how-to.md#pre-commit-hook

It seems like the Dockerfile is already written, I would think all you'd need is a docker version of each hook

MaxymVlasov mentioned this issue

fix: Dockerized pre-commit-terraform #219

Merged

MaxymVlasov self-assigned this

MaxymVlasov added estimate/2h feature labels

Collaborator

MaxymVlasov commented Sep 9, 2021 •

edited

Loading

Already done in GH-217/fix_docker branch. I suppose it will be merged in few days

MaxymVlasov closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment