Skip to content

Align website deploy with the crank/shovel CI flow#84

Open
brainkim wants to merge 1 commit into
masterfrom
deploy-flow
Open

Align website deploy with the crank/shovel CI flow#84
brainkim wants to merge 1 commit into
masterfrom
deploy-flow

Conversation

@brainkim

@brainkim brainkim commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

Makes the website deploy the same way crank and shovel do — from CI, not by hand.

What

  • CI deploy job (.github/workflows/main.yml): on push to master when website/ or docs/ change, a deploy job runs npm run deploy, which gh-pages-pushes dist/public to the gh-pages branch, authenticated with GITHUB_TOKEN. A changes job gates it; workflow_dispatch allows a manual deploy.
  • Website scripts now match shovel's shape: develop / static / predeploy / deploy.
  • Drops pagefind (search) — script, dependency, shovel.json mount, and the /pagefind route — matching shovel's simpler flow.

Notes

  • Stays on --platform node, not bun — bun mishandles unicode in the JSX template tags the site uses (curly quotes, arrows, emoji).
  • Merging this PR triggers the first deploy (it touches website/), replacing the 6-year-old Docusaurus site at repeater.js.org.
  • The deploy job needs Actions write access (it declares permissions: contents: write); if the repo defaults GITHUB_TOKEN to read-only, enable Settings → Actions → General → Workflow permissions → Read and write.

🤖 Generated with Claude Code

Deploy the site from CI on push to master (when website/ or docs/ change),
mirroring how crank and shovel deploy: a `deploy` job runs `npm run deploy`,
which gh-pages-pushes dist/public, authenticated with GITHUB_TOKEN. A `changes`
job gates it, and workflow_dispatch allows a manual deploy.

Stays on `--platform node` (not bun) because bun mishandles unicode in the JSX
template tags the site uses. Drops pagefind (search) — script, dependency,
shovel.json mount, and the /pagefind route — to match shovel's simpler flow.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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