Skip to content

fix(ci): only deploy the API to production from main - #54

Merged
cooper (czxtm) merged 1 commit into
mainfrom
fix/deploy-api-main-only
Sep 23, 2026
Merged

cooper (czxtm) merged 1 commit into
mainfrom
fix/deploy-api-main-only

Conversation

@czxtm

Copy link
Copy Markdown
Member

deploy-api.yaml also triggered on feat/cloud-gate-** pushes, and none of its build, push or flyctl deploy steps checked the branch. So a push to such a branch, or a manual dispatch from any branch, deployed to the production Fly app. The comment saying production still gated on main was wrong.

This PR drops the temporary branch glob and gates the job on refs/heads/main. It also removes the develop and pull_request conditions, which can no longer fire.

Verification: actionlint passes.

Refs stackpanel-thq.2

deploy-api.yaml also triggered on feat/cloud-gate-** pushes, and none of
its build, push or flyctl deploy steps checked the branch, so a push to
such a branch (or a manual dispatch from any branch) deployed to the
production Fly app. The comment claiming production still gated on main
was wrong.

Drop the temporary branch glob, gate the job on refs/heads/main, and
remove the develop/pull_request conditions that can no longer fire.

Refs: stackpanel-thq.2
@github-actions

github-actions Bot commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Preview pr-54 has been destroyed.

@czxtm
cooper (czxtm) merged commit 0f2717f into main Sep 23, 2026
13 of 14 checks passed
@linear-code

linear-code Bot commented Sep 23, 2026

Copy link
Copy Markdown

ENG-836

@github-actions

Copy link
Copy Markdown
Contributor

Docs preview pr-54 has been destroyed.

@czxtm
cooper (czxtm) deleted the fix/deploy-api-main-only branch September 23, 2026 12:50

This branch had an error being deployed

1 failed and 1 active deployments
pr-54 — 178ded7c Deployed Sep 23, 2026 by czxtm via deploy #194
docs-preview — 178ded7c Deployed Sep 23, 2026 by czxtm via deploy #179
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.

1 participant