From 3213eeb27313bcc2568473d2caa86a4f329cdf92 Mon Sep 17 00:00:00 2001 From: Austin McKinley Date: Tue, 12 May 2026 12:38:29 -0700 Subject: [PATCH] ci: build ogcard image on push to main The workflow only triggered on pull_request, which meant the image tagged with a PR commit SHA would get pushed to ECR, but the merge commit to main never produced a fresh image. Deploy tooling resolving main to an ECR tag would either miss or pick up a stale PR build. Match bskyweb-aws: `push: branches: [main]` + `workflow_dispatch`. Drops the pull_request trigger entirely. If Dockerfile validation on PRs is wanted back, we can add a hybrid pattern later (build-on-PR, push-only-on-main). Co-Authored-By: Claude Opus 4.7 (1M context) --- .github/workflows/build-and-push-ogcard-aws.yaml | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/.github/workflows/build-and-push-ogcard-aws.yaml b/.github/workflows/build-and-push-ogcard-aws.yaml index 961768b2b1..f58400b44f 100644 --- a/.github/workflows/build-and-push-ogcard-aws.yaml +++ b/.github/workflows/build-and-push-ogcard-aws.yaml @@ -1,11 +1,9 @@ name: build-and-push-ogcard-aws on: workflow_dispatch: - pull_request: - paths: - - "bskyogcard/**" - - "Dockerfile.bskyogcard" - - ".github/workflows/build-and-push-ogcard-aws.yaml" + push: + branches: + - main env: REGISTRY: ${{ secrets.AWS_ECR_REGISTRY_USEAST2_PACKAGES_REGISTRY }}