| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
github.event.head_commit.modified is only populated for single-commit pushes -- a squash-merge touching aws/cloudformation would silently skip infra sync. The CFN deploy is already idempotent (--no-fail-on-empty-changeset), so simplest robust fix is to just run it every time; costs one extra ~10-20s idempotent call per deploy.
if: always() made this job run even if the workflow was cancelled before it started. Now that deploy-infrastructure always runs (prior commit), it can never legitimately be 'skipped' either, so the implicit needs: gating (success-only, false on cancellation) is exactly the behavior wanted -- no explicit if: needed.
| Back | FazBrowse Home | New Git URL |
Summary
Phase 2 of the ECS CD-pipeline plan. Stacked on #108 — merge that first, this retargets to dev automatically.
Two independent correctness fixes in deploy-to-ecs.yml, found while reading it for the smoke-gate/rollback work that follows:
Test plan