fix: install deps for pre-commit

.github/workflows/pre-commit.yaml

@@ -14,5 +14,28 @@ jobs:
       - uses: actions/checkout@v4
       - name: install pre-commit
         run: pip install pre-commit
+
+      - name: Terraform
+        uses: hashicorp/setup-terraform@v3
+        with:
+          terraform_version: "1.5.7"
+
+      - name: install hcledit
+        run: |
+          curl -sSLo ./hcledit.tar.gz https://github.com/minamijoyo/hcledit/releases/download/v0.2.10/hcledit_0.2.10_linux_amd64.tar.gz
+          tar -xzf hcledit.tar.gz
+          chmod +x hcledit
+          mv hcledit /usr/local/bin/hcledit
+
+      - name: install tf-lint
+        run: curl -s https://raw.githubusercontent.com/terraform-linters/tflint/master/install_linux.sh | bash
+
+      - name: install tf-docs
+        run: |
+          curl -sSLo ./terraform-docs.tar.gz https://terraform-docs.io/dl/v0.17.0/terraform-docs-v0.17.0-linux-amd64.tar.gz
+          tar -xzf terraform-docs.tar.gz
+          chmod +x terraform-docs
+          mv terraform-docs /usr/local/bin/terraform-docs
+
       - name: pre-commit
         run: pre-commit run --all-files --show-diff-on-failure

.pre-commit-config.yaml

@@ -22,22 +22,22 @@ repos:
   hooks:
     - id: terraform_fmt
     - id: terraform_wrapper_module_for_each
-#    - id: terraform_validate
+    - id: terraform_validate
     - id: terraform_docs
       args:
         - '--args=--lockfile=false'
     - id: terraform_tflint
-#      args:
-#        - '--args=--only=terraform_deprecated_interpolation'
-#        - '--args=--only=terraform_deprecated_index'
-#        - '--args=--only=terraform_unused_declarations'
-#        - '--args=--only=terraform_comment_syntax'
-#        - '--args=--only=terraform_documented_outputs'
-#        - '--args=--only=terraform_documented_variables'
-#        - '--args=--only=terraform_typed_variables'
-#        - '--args=--only=terraform_module_pinned_source'
-#        - '--args=--only=terraform_naming_convention'
-#        - '--args=--only=terraform_required_version'
-#        - '--args=--only=terraform_required_providers'
-#        - '--args=--only=terraform_standard_module_structure'
-#        - '--args=--only=terraform_workspace_remote'
+      args:
+        - '--args=--only=terraform_deprecated_interpolation'
+        - '--args=--only=terraform_deprecated_index'
+        - '--args=--only=terraform_unused_declarations'
+        - '--args=--only=terraform_comment_syntax'
+        - '--args=--only=terraform_documented_outputs'
+        - '--args=--only=terraform_documented_variables'
+        - '--args=--only=terraform_typed_variables'
+        - '--args=--only=terraform_module_pinned_source'
+        - '--args=--only=terraform_naming_convention'
+        - '--args=--only=terraform_required_version'
+        - '--args=--only=terraform_required_providers'
+        - '--args=--only=terraform_standard_module_structure'
+        - '--args=--only=terraform_workspace_remote'

terraform/example/main.tf

@@ -21,9 +21,6 @@ provider "aws" {
   region = "us-west-2"
 }
 data "aws_caller_identity" "current" {}
-output "account_id" {
-  value = data.aws_caller_identity.current.account_id
-}
 resource "aws_ebs_encryption_by_default" "us_west_2" {
   enabled = true
 }
@@ -163,7 +160,7 @@ module "karpenter" {
   node_iam_role_additional_policies = {
     AmazonSSMManagedInstanceCore = "arn:aws:iam::aws:policy/AmazonSSMManagedInstanceCore"
   }
-  // TODO: get a better naming conventions for roles
+  # TODO: get a better naming conventions for roles
   node_iam_role_name            = "KarpenterNodeRole-${local.core_cluster_name}"
   iam_role_name                 = "${local.core_cluster_name}-karpenter-role"
   iam_role_use_name_prefix      = false

terraform/example/outputs.tf

@@ -0,0 +1,5 @@
+output "account_id" {
+  description = "sample account id"
+  value       = data.aws_caller_identity.current.account_id
+
+}

terraform/example/variables.tf

@@ -1,8 +1,10 @@
 variable "owner" {
-  default = "pelotech"
-  type    = string
+  description = "owner tags which get added to the resources"
+  default     = "pelotech"
+  type        = string
 }
 variable "environment" {
-  default = "production"
-  type    = string
+  description = "environment tags which get added to the resources"
+  default     = "production"
+  type        = string
 }