fix: Add error handling around directories (#16)

Co-authored-by: Bryant Biggs <[email protected]>

.github/workflows/pre-commit.yml

@@ -41,7 +41,7 @@ jobs:
 
  - name: Terraform min/max versions
  id: minMax
- uses: clowdhaus/[email protected].3
+ uses: clowdhaus/[email protected].4
  with:
  directory: ${{ matrix.directory }}
 
@@ -81,7 +81,7 @@ jobs:
 
  - name: Terraform min/max versions
  id: minMax
- uses: clowdhaus/[email protected].3
+ uses: clowdhaus/[email protected].4
 
  - name: Pre-commit Terraform ${{ steps.minMax.outputs.maxVersion }}
  uses: ./pre-commit

directories/action.yml

@@ -18,18 +18,22 @@ runs:
  import os
  import re
  import glob
+ from pathlib import Path
 
 
  def get_directories():
  """
  Return all Terraform (*.tf) files that contain the 'required_version'
- string, but does not contain 'wrapper' in the path.
+  string, but does not contain 'wrapper' in the path.
  """
 
  terraform_files = glob.glob('./**/*.tf', recursive=True)
  directories = []
 
  for file in terraform_files:
+ p = Path(file)
+ if p.is_dir():
+ continue
  file_size = os.stat(file)
 
  if file_size.st_size > 0:

directories/directories.py

@@ -3,6 +3,7 @@
 import os
 import re
 import glob
+from pathlib import Path
 
 
 def get_directories():
@@ -15,6 +16,9 @@ def get_directories():
  directories = []
 
  for file in terraform_files:
+ p = Path(file)
+ if p.is_dir():
+ continue
  file_size = os.stat(file)
 
  if file_size.st_size > 0:

examples/cost.modules.tf/main.tf

@@ -0,0 +1 @@
+locals {}

examples/cost.modules.tf/versions.tf

@@ -0,0 +1,14 @@
+provider "aws" {
+ region = "us-east-1"
+}
+
+terraform {
+ required_version = ">= 1.0.0"
+
+ required_providers {
+ aws = {
+ source = "hashicorp/aws"
+ version = ">= 4.0"
+ }
+ }
+}