Skip to content

[_]: chore/upgrade Node.js to 24 - #2179

Merged
terrerox merged 6 commits into
masterfrom
chore/upgrade-node-24
Oct 7, 2026
Merged

terrerox merged 6 commits into
masterfrom
chore/upgrade-node-24

Conversation

@terrerox

@terrerox terrerox commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Description

Upgrades Drive Web's development and production environments from Node.js 22 to Node.js 24.

  • CI: ci.yml (3 jobs), playwright.yml and sonar-analysis.yml now run on 24.x. Sonar also had a naming mismatch (node_version in the matrix vs node-version in the step), so it was silently using the runner's default Node. That is fixed.
  • Docker: development, preview, staging and share images now use node:24-alpine. Development was on the floating lts tag, preview/share on 18 and staging on 16.
  • Added .nvmrc (24) so local environments match CI and Docker.

Related Issues

Related Pull Requests

Checklist

  • Changes have been tested locally.
  • Unit tests have been written or updated as necessary.
  • The code adheres to the repository's coding standards.
  • Relevant documentation has been added or updated.
  • No new warnings or errors have been introduced.
  • SonarCloud issues have been reviewed and addressed.
  • QA Passed

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

Deploying drive-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: ff43d0c
Status: ✅  Deploy successful!
Preview URL: https://df5ac60e.drive-web.pages.dev
Branch Preview URL: https://chore-upgrade-node-24.drive-web.pages.dev

View logs

@terrerox
terrerox marked this pull request as ready for review September 25, 2026 01:50
@sonarqubecloud

sonarqubecloud Bot commented Oct 6, 2026

Copy link
Copy Markdown

@terrerox terrerox self-assigned this Oct 7, 2026
@terrerox
terrerox merged commit 4a82713 into master Oct 7, 2026
11 checks passed
@terrerox
terrerox deleted the chore/upgrade-node-24 branch October 7, 2026 13: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