Skip to content

Conversation

@jamesiarmes
Copy link
Member

No description provided.

@jamesiarmes jamesiarmes requested a review from Copilot October 9, 2025 17:29
@jamesiarmes jamesiarmes marked this pull request as ready for review October 9, 2025 17:29
@jamesiarmes jamesiarmes requested a review from a team as a code owner October 9, 2025 17:29
Copy link

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 refactors GitHub Actions workflows to reduce duplication and improve maintainability by consolidating common functionality into reusable actions and separate workflow files.

  • Consolidates security tooling (TFLint and Trivy) into separate dedicated workflows that can run on both push and pull request events
  • Extracts common module discovery logic into a reusable composite action
  • Creates a reusable action for checking GitHub security features to conditionally enable SARIF uploads

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
.github/workflows/trivy.yaml New dedicated workflow for Trivy security scanning with conditional SARIF upload
.github/workflows/tflint.yaml New dedicated workflow for TFLint checks with conditional SARIF upload
.github/workflows/pull-request.yaml Simplified to use new changed-modules action instead of inline logic
.github/workflows/main.yaml Removed - functionality moved to dedicated tool workflows
.github/workflows/branch.yaml Removed - functionality consolidated into other workflows
.github/actions/security-features/action.yaml New composite action to check repository security capabilities
.github/actions/changed-modules/action.yaml New composite action to discover changed OpenTofu modules

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@codeforamerica codeforamerica deleted a comment from github-actions bot Oct 9, 2025
@jamesiarmes jamesiarmes merged commit f598389 into main Oct 9, 2025
9 checks passed
@jamesiarmes jamesiarmes deleted the ci-rework branch October 9, 2025 17:33
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