Skip to content

chore(ci): record the demo alongside test and deploy - #10

Merged
sam-goodwin merged 1 commit into
chore/ci-cleanup-never-cancelledfrom
chore/ci-demo-parallel
Oct 7, 2026
Merged

sam-goodwin merged 1 commit into
chore/ci-cleanup-never-cancelledfrom
chore/ci-demo-parallel

Conversation

@sam-goodwin

Copy link
Copy Markdown
Collaborator

Stacked on #9.

The demo job waited for test and deploy (~6 min), but it deploys its own stages (demo-, dev-, test-demo-pr-<n>) and only needed the preview URL for its comment.

before: check → test → deploy → demo (comment)
after:  check → test → deploy ─┐
             └→ demo ──────────┴→ demo-comment
  demo:
    needs: check
    outputs:
      url: ${{ steps.upload.outputs.url }}

  demo-comment:
    if: github.event_name == 'pull_request'
    needs: [deploy, demo]
    runs-on: ubuntu-latest
    steps:
      - run: bun packages/demos/comment.ts
        env:
          DEMO_URL: ${{ needs.demo.outputs.url }}
          APP_URL: ${{ needs.deploy.outputs.url }}

The macOS runner queue still applies, but it now overlaps with test and deploy instead of following them.

🤖 Generated with Claude Code

The demo deploys its own stages, so it only needs check. The PR comment,
which links the demo and the preview, moves to a demo-comment job that
waits for both.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown

Preview deployed

App: https://cafe-web-pr-10-szezdpjidwn3zqed.testing-2b2.workers.dev

Built from commit 8063e85.


This comment updates automatically with each push.

@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown

Demo of this preview

Demo of this preview

Video: https://pub-68e74efbba3b4272943c198da363bcab.r2.dev/pr-10/3404b26cf05b76774fe63dde2e1ccec08d6a2f10/demo.mp4
App: https://cafe-web-pr-10-szezdpjidwn3zqed.testing-2b2.workers.dev

Recorded from commit 3404b26: alchemy deploy, chatting in the browser, alchemy dev, pnpm test, ALCHEMY_DEV=1 pnpm test.

@sam-goodwin
sam-goodwin added this pull request to stack #11 October 7, 2026 09:39
@sam-goodwin
sam-goodwin merged commit c35b099 into main Oct 7, 2026
5 checks passed
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