Skip to content

ci: run the Netlify CLI on Node 24 - #1060

Merged
mmcky merged 1 commit into
mainfrom
maint/node24-actions
Sep 18, 2026
Merged

mmcky merged 1 commit into
mainfrom
maint/node24-actions

Conversation

@mmcky

@mmcky mmcky commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

The preview job in ci.yml sets up Node 20 for the Netlify deploy. This PR moves it to Node 24, which is the active LTS line. Node 20 reached end-of-life on 2026-04-30.

Node runs only the Netlify CLI here, which quantecon/actions/preview-netlify installs with npm install -g netlify-cli@latest. netlify-cli 27.0.0 (released 2026-07-24) raised its engine floor to Node >=22.13.0, and the current release (27.8.0) keeps it. So the preview deploy has been installing the CLI outside its supported range since late July. A recent preview run shows it: run 35283257505 on 2026-09-17 logged npm warn EBADENGINE for netlify-cli@27.8.0 (required >=22.13.0, current v20.20.2), and for several @netlify/* dependencies. npm warns but installs anyway, so the deploys have kept working.

Pin From To Runtime
node-version for actions/setup-node@v7 (ci.yml, Setup Node.js) '20' '24' the job's own Node, used by the Netlify CLI
actions/setup-node@v7 unchanged unchanged node24
quantecon/actions/preview-netlify@v0 unchanged unchanged composite, whose only inner action is actions/github-script@v9 (node24)

No uses: line changes. This is one line in one file.

What CI proves. This PR's own CI runs the changed workflow: ci.yml runs on pull_request, and because this branch is in the same repository and not from Dependabot, the Install Netlify CLI and Deploy to Netlify steps run on Node 24. The warnings should be gone from that step's log. Nothing is left to prove after merge, because no other workflow uses Node.

This is part of the Node 24 upgrade across the lecture and theme fleets, approved on 2026-09-18.

🤖 Generated with Claude Code

The preview job's setup-node step pinned node-version '20'. Node 20
reached end-of-life on 2026-04-30, and netlify-cli 27.0.0 (2026-07-24)
raised its engine floor to Node >=22.13.0, so the preview deploy has
been installing netlify-cli@latest outside its supported range (npm
logs EBADENGINE). Node here runs only the Netlify CLI inside
quantecon/actions/preview-netlify.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings September 18, 2026 07:23

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The change is a single, correctly-indented workflow update that aligns CI’s Node runtime with Netlify CLI’s supported engine range.

Pull request overview

Updates the GitHub Actions CI workflow to use Node.js 24 for the Netlify preview deploy, aligning the workflow runtime with the current Netlify CLI engine requirements and avoiding EBADENGINE warnings.

Changes:

  • Bump actions/setup-node@v7 node-version in the preview deploy job from '20' to '24'.
File summaries
File Description
.github/workflows/ci.yml Updates the Node.js version used for the Netlify preview deploy step to Node 24.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@github-actions

Copy link
Copy Markdown

📖 Netlify Preview Ready!

Preview URL: https://pr-1060--sunny-cactus-210e3e.netlify.app

Commit: 7ba7c65


Build Info

@mmcky
mmcky merged commit f1ad055 into main Sep 18, 2026
3 checks passed
@mmcky
mmcky deleted the maint/node24-actions branch September 18, 2026 11:25
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.

2 participants