Skip to content

chore: Raise the Node engines floor to 24 - #43

Merged
ecv merged 1 commit into
mainfrom
chore/node-24-floor
Nov 5, 2026
Merged

ecv merged 1 commit into
mainfrom
chore/node-24-floor

Conversation

@ecv

@ecv ecv commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

The CMS's engines range still admitted Node 22, which loses upstream support in April 2027. Strapi Cloud hosts the CMS, and Node 24 is the version we accept there, so the floor moves to 24. Images and CI stay on Node 26 via .nvmrc.

Until now CI only built on Node 26, so nothing showed the CMS actually installs and builds on the floor version. The required check now does both.

Changes

  • engines.node is >=24.0.0 <=26.x.x in package.json and package-lock.json.
  • The required CI check also runs npm ci and npm run build on Node 24, and fails if the Node it runs is not 24.

Not covered here

  • The Strapi Cloud Node setting (dashboard, General tab) is not visible from the repo. .npmrc sets no engine-strict, so if Strapi Cloud still runs Node 22, npm only warns at install and the deploy does not fail. Someone with Strapi Cloud access should confirm it is on Node 24.

Related to #34
Related to https://github.com/datum-cloud/infra/issues/8215

Strapi Cloud is the CMS host and Node 24 is the version we accept there,
so the engines range no longer needs to admit Node 22. CI built only on
the .nvmrc Node 26, so nothing proved the floor actually works.

Key changes:
- engines.node is >=24.0.0 <=26.x.x in package.json and the lockfile
- CI also installs and builds on Node 24, failing if the Node it runs is
  not 24
@cla-assistant

cla-assistant Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@ecv
ecv requested a review from andrewb3000 November 5, 2026 01:31
@ecv
ecv enabled auto-merge November 5, 2026 01:31
@ecv
ecv merged commit 72d3621 into main Nov 5, 2026
2 checks passed
@ecv
ecv deleted the chore/node-24-floor branch November 5, 2026 07: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