Skip to content

fix(pages): self-enable Pages in the deploy workflow (first-run 404) - #28

Merged
liujonathan24 merged 1 commit into
mainfrom
wasm-pages-fix
Jul 15, 2026
Merged

fix(pages): self-enable Pages in the deploy workflow (first-run 404)#28
liujonathan24 merged 1 commit into
mainfrom
wasm-pages-fix

Conversation

@liujonathan24

Copy link
Copy Markdown
Owner

Follow-up to #27. The WASM site is fully merged, but the GitHub Pages deploy failed on its first run (the #27 merge) at actions/configure-pages@v5:

Get Pages site failed. Please verify that the repository has Pages enabled and configured to build using GitHub Actions... Error: Not Found

This is the classic first-run chicken-and-egg: the action tried to read the Pages site before Pages was enabled with the GitHub Actions build type, got a 404, and aborted. Nothing was missing from #27 — the deploy just never retried (it only re-runs on a push to deploy/wasm-web/** or the workflow file).

Fix

Add enablement: true to the configure-pages step so it enables Pages via the API if it isn't already, making the workflow self-sufficient and idempotent:

      - uses: actions/configure-pages@v5
        with:
          enablement: true

What happens on merge

Merging this touches the workflow path → re-triggers the deploy → configure-pages now passes → deploy-pages publishes deploy/wasm-web/ to https://liujonathan24.github.io/NetHackHarness/.

(The Node-20 deprecation warnings in the failed log are non-fatal; the action versions used are already current.)

https://claude.ai/code/session_01D3HuJeFjkPti8RgmT88joG

The GitHub Pages deploy failed on the first run (the #27 merge) at
actions/configure-pages@v5: "Get Pages site failed ... Error: Not Found" — the
action read the Pages site before Pages was enabled with the GitHub Actions build
type. Add `enablement: true` so configure-pages turns Pages on via the API if it
isn't already, making the workflow self-sufficient and idempotent.

Merging this re-triggers the workflow (touches the workflow path), which then
publishes deploy/wasm-web/ to https://liujonathan24.github.io/NetHackHarness/.

Claude-Session: https://claude.ai/code/session_01D3HuJeFjkPti8RgmT88joG
@liujonathan24
liujonathan24 merged commit 5361f93 into main Jul 15, 2026
1 of 2 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