Skip to content

ci(demo): build the demo on pull requests too - #6

Merged
LukiPrince merged 1 commit into
mainfrom
ci/demo-on-pull-requests
Nov 7, 2026
Merged

LukiPrince merged 1 commit into
mainfrom
ci/demo-on-pull-requests

Conversation

@LukiPrince

Copy link
Copy Markdown
Owner

The demo workflow only ran on main, so a step that breaks there showed up after the merge (#4).

  • Pull requests that touch web/ or .github/workflows/demo.yml now run the same steps; only the wrangler deploy is limited to main (on a pull request github.ref is refs/pull/<n>/merge).
  • A pull request gets its own concurrency group, so it can never take the place of a deploy that is waiting.
  • The workflow file itself is in paths, so this pull request runs it, and the merge redeploys the demo once.

🤖 Generated with Claude Code

https://claude.ai/code/session_01TEvvb1DM3ZA5FqELYSs4v8

The demo workflow only ran on main, so a step that breaks there (#4) showed up after the merge.
Pull requests that touch web/ or this workflow now run the same steps; the deploy itself stays
on main. A pull request gets its own concurrency group, so it never cancels a waiting deploy.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TEvvb1DM3ZA5FqELYSs4v8
@LukiPrince
LukiPrince merged commit 51a9675 into main Nov 7, 2026
27 checks passed
@LukiPrince
LukiPrince deleted the ci/demo-on-pull-requests branch November 7, 2026 17:26
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