ran pre-commit and added new terraform validation checks

.editorconfig

@@ -15,7 +15,7 @@ max_line_length = 80
 trim_trailing_whitespace = true
 
 [*.{yaml,yml}]
-max_line_length = 120
+max_line_length = 180
 
 [*.{tf,tfvars}]
 indent_size = 2

.github/workflows/pipeline.yaml

@@ -1,5 +1,6 @@
+---
 name: Terraform
-on:
+on:    # yamllint disable-line rule:truthy
   push:
     branches: [main]
   pull_request:

.github/workflows/tag-release.yml

@@ -20,7 +20,7 @@ jobs:
         id: create_release
         uses: actions/[email protected]
         env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # This token is provided by Actions, you do not need to create your own token
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         with:
           tag_name: ${{ steps.tag.outputs.tag }}
           release_name: v${{ steps.tag.outputs.tag }}

.pre-commit-config.yaml

@@ -5,7 +5,7 @@ repos:
     hooks:
       - id: yamllint
         args:
-          - '-d {extends: default, rules: {line-length: {max: 120}}}'  # override to match .editorconfig
+          - '-d {extends: default, rules: {line-length: {max: 180}}}'  # override to match .editorconfig
           - -s
   - repo: https://github.com/pre-commit/pre-commit-hooks
     rev: v4.4.0

example/README.md

@@ -10,7 +10,9 @@
 
 ## Providers
 
-No providers.
+| Name | Version |
+|------|---------|
+| <a name="provider_aws"></a> [aws](#provider\_aws) | 4.67.0 |
 
 ## Modules
 
@@ -21,7 +23,9 @@ No providers.
 
 ## Resources
 
-No resources.
+| Name | Type |
+|------|------|
+| [aws_caller_identity.current](https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/caller_identity) | data source |
 
 ## Inputs
 

example/locals.tf

@@ -15,13 +15,13 @@ locals {
         ],
         Resource = "*",
       },
-      # {
-      #   Sid       = "AllowKeyOwnerToUpdate",
-      #   Effect    = "Allow",
-      #   Principal = { AWS = data.aws_caller_identity.current.account_id },  // if required then use it
-      #   Action    = "kms:*",
-      #   Resource  = "*",
-      # },
+      {
+        Sid       = "AllowKeyOwnerToUpdate",
+        Effect    = "Allow",
+        Principal = { AWS = data.aws_caller_identity.current.account_id }, // if required then use it
+        Action    = "kms:*",
+        Resource  = "*",
+      },
     ],
   })
 }

example/main.tf

@@ -1,4 +1,4 @@
-# data "aws_caller_identity" "current" {}
+data "aws_caller_identity" "current" {}
 module "tags" {
   source  = "sourcefuse/arc-tags/aws"
   version = "1.2.3"

example/mock_provider.tf

@@ -10,7 +10,7 @@ terraform {
 }
 
 provider "aws" {
-  region                      = "us-east-1"
+  region                      = var.region
   skip_credentials_validation = true
   skip_requesting_account_id  = true
   skip_metadata_api_check     = true
@@ -38,4 +38,4 @@ provider "aws" {
     sts            = "http://localstack:4566"
     kms            = "http://localstack:4566"
   }
-}
+}