Skip to content

Commit

Permalink
style(terraform): move comment
Browse files Browse the repository at this point in the history
  • Loading branch information
hknutsen committed Feb 12, 2025
1 parent da84a85 commit 1ec090b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/terraform.yml
Original file line number Diff line number Diff line change
Expand Up @@ -236,8 +236,6 @@ jobs:
path: ${{ steps.mkdir.outputs.plugin-cache-dir }}
key: ${{ steps.cache-restore.outputs.cache-primary-key }}

# Some plans are too large to be stored in environment variables or step outputs, resulting in an error.
# To bypass this, the plan must be explicitly read during this step using the "terraform show" command.
- name: Create job summary
# Only run if Terraform Plan succeeded with non-empty diff (changes present).
# Ref: https://developer.hashicorp.com/terraform/cli/commands/plan#detailed-exitcode
Expand All @@ -249,6 +247,8 @@ jobs:
TF_VALIDATE_OUTCOME: ${{ steps.validate.outcome }}
TF_PLAN_OUTCOME: ${{ steps.plan.outcome }}
WORKING_DIRECTORY: ${{ inputs.working_directory }}
# Some plans are too large to be stored in environment variables or step outputs, resulting in an error.
# To bypass this, the plan must be explicitly read during this step using the "terraform show" command.
run: |
tfplan=$(terraform show "$TF_PLAN_FILE" -no-color)
Expand Down

0 comments on commit 1ec090b

Please sign in to comment.