Skip to content

Commit

Permalink
Merge pull request #667 from heroku/fix-removal-workflow-dry-run-output
Browse files Browse the repository at this point in the history
Fix output heading for package removal workflow when using dry-run mode
  • Loading branch information
dzuelke authored Dec 7, 2023
2 parents c005882 + 50ea36d commit 2ab4fca
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/platform-remove.yml
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ jobs:
(yes 2>/dev/null || true) | docker run --rm -i --env-file=support/build/_docker/env.default heroku-php-build-${{inputs.stack}}:${{github.sha}} remove.sh ${{inputs.manifests}} 2>&1 | tee remove.out
- name: Output job summary
run: |
echo '## Packages${{ inputs.dry-run == true && ' which would be' }} removed from production bucket' >> "$GITHUB_STEP_SUMMARY"
echo '## Packages${{ inputs.dry-run == true && ' which would be' || '' }} removed from production bucket' >> "$GITHUB_STEP_SUMMARY"
echo "${{ inputs.dry-run == true && '**This is output from a dry-run**, no packages have been removed:' || '-n' }}" >> "$GITHUB_STEP_SUMMARY"
echo '```' >> "$GITHUB_STEP_SUMMARY"
sed -n '/The following packages will/,$p' remove.out >> "$GITHUB_STEP_SUMMARY"
Expand Down

0 comments on commit 2ab4fca

Please sign in to comment.