Skip to content

Deploy to Cloudflare Pages on every push to main - #1

Closed
nedtwigg wants to merge 2 commits into
mainfrom
deploy-on-push
Closed

nedtwigg wants to merge 2 commits into
mainfrom
deploy-on-push

Conversation

@nedtwigg

@nedtwigg nedtwigg commented Nov 5, 2026

Copy link
Copy Markdown
Member

Today a push only updates GitHub, because selfie-landing is a direct-upload Pages project. This adds a workflow that, on every push to main, runs the tests, deploys public/, and runs the tests again against https://selfie.dev. Pull requests only run the tests.

It also pins the pnpm version, and updates the README now that selfie.dev has moved.

Before merging: add a CLOUDFLARE_API_TOKEN repository secret with the Cloudflare Pages Edit permission. Without it, the deploy job fails on merge. The live site stays as it is either way.

The Cloudflare account ID is in the workflow on purpose. It isn't a credential, and it already appears in dashboard URLs.

🤖 Generated with Claude Code

nedtwigg and others added 2 commits November 4, 2026 17:06
Pushes used to update only GitHub, because selfie-landing is a direct-upload
project. Tests run first, and again against selfie.dev after the deploy, so
a broken redirect fails the run. Also update the README now that the domain
has moved.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The pinned pnpm version has to be in the lockfile, and pnpm fails the install
while esbuild's and workerd's install scripts are undecided. Wrangler works
without them, so they're denied.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@nedtwigg

nedtwigg commented Nov 5, 2026

Copy link
Copy Markdown
Member Author

We don't need autodeploy for now.

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