Skip to content

Upgrade Node.js to v22 in CI workflows#882

Merged
willtheorangeguy merged 1 commit intomainfrom
copilot/fix-unsupported-node-version
Mar 24, 2026
Merged

Upgrade Node.js to v22 in CI workflows#882
willtheorangeguy merged 1 commit intomainfrom
copilot/fix-unsupported-node-version

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 24, 2026

Astro v6 dropped support for Node.js <22.12.0, causing the GitHub Pages build to fail on the previously pinned v20 runner.

Changes

  • .github/workflows/astro.ymlnode-version: "20""22"
  • .github/workflows/update-google-maps-stats.ymlnode-version: '20''22'

@willtheorangeguy willtheorangeguy marked this pull request as ready for review March 24, 2026 12:59
Copilot AI review requested due to automatic review settings March 24, 2026 12:59
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates GitHub Actions workflows to use Node.js 22 to restore compatibility with Astro v6 (which no longer supports Node.js < 22.12.0) and unblock the GitHub Pages build.

Changes:

  • Bumped Node.js version in the Astro Pages deployment workflow from 20 → 22.
  • Bumped Node.js version in the Google Maps stats update workflow from 20 → 22.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.github/workflows/astro.yml Updates Node.js version used for the Astro build/deploy job.
.github/workflows/update-google-maps-stats.yml Updates Node.js version used for the scheduled stats update job.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

uses: actions/setup-node@v6
with:
node-version: "20"
node-version: "22"
Copy link

Copilot AI Mar 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Astro v6 requires Node.js >= 22.12.0 (per PR description). node-version: "22" only pins the major and can resolve to an earlier 22.x release depending on what setup-node selects, which could reintroduce the build failure. Consider pinning to at least 22.12.0 (or 22.12.x) to enforce the minimum supported version deterministically.

Suggested change
node-version: "22"
node-version: "22.12.0"

Copilot uses AI. Check for mistakes.
uses: actions/setup-node@v6
with:
node-version: '20'
node-version: '22'
Copy link

Copilot AI Mar 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Astro v6 requires Node.js >= 22.12.0 (per PR description). node-version: '22' only pins the major and may resolve to an earlier 22.x release, which could break scripts if they assume the minimum supported version. Consider pinning to 22.12.0 (or 22.12.x) here as well for deterministic workflow behavior.

Suggested change
node-version: '22'
node-version: '22.12.0'

Copilot uses AI. Check for mistakes.
@willtheorangeguy willtheorangeguy merged commit 53f1146 into main Mar 24, 2026
13 of 14 checks passed
@willtheorangeguy willtheorangeguy deleted the copilot/fix-unsupported-node-version branch March 24, 2026 13:02
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.

3 participants