Skip to content

Conversation

@kurtismash
Copy link
Member

No description provided.

@kurtismash kurtismash requested a review from Copilot July 19, 2025 14:12
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR renames the GitHub Actions workflow from "Infrastructure Checks" to "Lint and Validate Terraform" to better reflect its specific purpose. The changes focus on improving naming consistency and adding security permissions.

  • Renamed workflow and updated file references to match new naming convention
  • Standardized step names to use lowercase and more descriptive formatting
  • Added explicit permissions for contents and checks access
Comments suppressed due to low confidence (3)

.github/workflows/lint-and-validate-terraform.yml:32

  • [nitpick] The step name should describe the action being performed rather than the command itself. Consider using 'Check Terraform formatting' or 'Validate Terraform format' instead of the raw command.
      - name: terraform fmt -check=true -recursive

.github/workflows/lint-and-validate-terraform.yml:34

  • [nitpick] The step name should describe the action being performed rather than the command itself. Consider using 'Initialize Terraform (no backend)' or 'Terraform initialization' instead of the raw command.
      - name: terraform init -backend=false

.github/workflows/lint-and-validate-terraform.yml:36

  • [nitpick] The step name should describe the action being performed rather than the command itself. Consider using 'Validate Terraform configuration' or 'Run Terraform validation' instead of the raw command.
      - name: terraform validate

@kurtismash kurtismash merged commit 96d7115 into main Jul 19, 2025
9 checks passed
@kurtismash kurtismash deleted the rename-action branch July 19, 2025 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants