mirror of
https://github.com/cisagov/log4j-affected-db.git
synced 2024-11-22 16:40:48 +00:00
Merge pull request #90 from cisagov/improvement/enable_terraform_validate
Enable terraform validate pre-commit hook
This commit is contained in:
commit
1b5cd2546b
2 changed files with 1 additions and 23 deletions
7
.github/workflows/build.yml
vendored
7
.github/workflows/build.yml
vendored
|
@ -84,13 +84,6 @@ jobs:
|
|||
run: |
|
||||
go install \
|
||||
github.com/terraform-docs/terraform-docs@${TERRAFORM_DOCS_VERSION}
|
||||
- name: Find and initialize Terraform directories
|
||||
run: |
|
||||
for path in $(find . -not \( -type d -name ".terraform" -prune \) \
|
||||
-type f -iname "*.tf" -exec dirname "{}" \; | sort -u); do \
|
||||
echo "Initializing '$path'..."; \
|
||||
terraform init -input=false -backend=false "$path"; \
|
||||
done
|
||||
- name: Install dependencies
|
||||
run: |
|
||||
python -m pip install --upgrade pip
|
||||
|
|
|
@ -117,22 +117,7 @@ repos:
|
|||
rev: v1.50.0
|
||||
hooks:
|
||||
- id: terraform_fmt
|
||||
# There are ongoing issues with how this command works. This issue
|
||||
# documents the core issue:
|
||||
# https://github.com/hashicorp/terraform/issues/21408
|
||||
# We have seen issues primarily with proxy providers and Terraform code
|
||||
# that uses remote state. The PR
|
||||
# https://github.com/hashicorp/terraform/pull/24887
|
||||
# has been approved and is part of the 0.13 release to resolve the issue
|
||||
# with remote states.
|
||||
# The PR
|
||||
# https://github.com/hashicorp/terraform/pull/24896
|
||||
# is a proprosed fix to deal with `terraform validate` with proxy
|
||||
# providers (among other configurations).
|
||||
# We have decided to disable the terraform_validate hook until the issues
|
||||
# above have been resolved, which we hope will be with the release of
|
||||
# Terraform 0.13.
|
||||
# - id: terraform_validate
|
||||
- id: terraform_validate
|
||||
|
||||
# Docker hooks
|
||||
- repo: https://github.com/IamTheFij/docker-pre-commit
|
||||
|
|
Loading…
Reference in a new issue