Skip to content

Use Node 24 for Pages deployment - #38

Merged
jmchilton merged 1 commit into
mainfrom
codex/fix-pages-node24
Aug 4, 2026
Merged

Use Node 24 for Pages deployment#38
jmchilton merged 1 commit into
mainfrom
codex/fix-pages-node24

Conversation

@jmchilton

Copy link
Copy Markdown
Member

What changed

  • configure withastro/action@v3 to build the Pages site with Node 24

Why

The Pages deployment still used the action's Node 20 default. The site now requires Node >=22.12.0, and Astro 7.1.6 refuses to build under Node 20. The post-merge deployment failed before Astro could render any pages.

Node 24 matches the toolchain documented by the project and satisfies both the site's engine constraint and Astro's supported range.

Impact

GitHub Pages builds use the same supported Node major expected by the repository. No site source or generated output changes.

Validation

  • clean npm ci
  • npm run build
  • npm test — 22 passed
  • npm run typecheck — no errors; one existing unused-variable hint
  • git diff --check

Fixes the failure in Pages run 30909495844.

@jmchilton
jmchilton marked this pull request as ready for review August 4, 2026 12:58
@jmchilton
jmchilton merged commit dc9959b into main Aug 4, 2026
1 check 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