Skip to content

Dry run action + workflow cleanup#2

Merged
randyquaye merged 6 commits intomainfrom
rq/imprv
Feb 11, 2026
Merged

Dry run action + workflow cleanup#2
randyquaye merged 6 commits intomainfrom
rq/imprv

Conversation

@randyquaye
Copy link
Collaborator

Added a new dry run workflow for terraform

Removed unnecessary deployment action

Copilot AI review requested due to automatic review settings February 11, 2026 17:12
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 modernizes the deployment workflow by introducing a dry-run capability for validating Terraform changes before merging, and cleaning up the deployment workflow by replacing the third-party deployment tracking action with GitHub's native environment protection.

Changes:

  • Added a new deploy-dry-run.yml workflow that runs on pull requests and workflow_dispatch events to validate Terraform changes without applying them
  • Removed the chrnorm/deployment-action and chrnorm/deployment-status steps from the deploy workflow in favor of native GitHub environment protection
  • Added environment: prod to the deploy job to enable GitHub's native deployment tracking and protection features
  • Removed the deployments: write permission which is no longer needed

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/deploy.yml Removed third-party deployment tracking actions and associated deployments: write permission; added native environment: prod for better GitHub integration
.github/workflows/deploy-dry-run.yml New workflow that runs Terraform plan on PRs to validate infrastructure changes without applying them; builds Lambda layer locally but skips S3 upload and terraform apply steps

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@randyquaye randyquaye merged commit 40714c7 into main Feb 11, 2026
4 of 5 checks passed
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.

3 participants