Skip to content

Bump E2E workflow Node.js to 22#1049

Merged
donnchawp merged 1 commit into
trunkfrom
fix-e2e-node-version
May 19, 2026
Merged

Bump E2E workflow Node.js to 22#1049
donnchawp merged 1 commit into
trunkfrom
fix-e2e-node-version

Conversation

@donnchawp
Copy link
Copy Markdown
Contributor

Summary

  • pnpm/action-setup@v4 with version: latest now pulls pnpm 11.1.3, which requires Node.js v22.13+ and imports node:sqlite (a built-in only available from Node 22).
  • actions/setup-node@v4 crashes with ERR_UNKNOWN_BUILTIN_MODULE while setting up the pnpm cache, failing every E2E run since Bump axios from 1.15.0 to 1.15.2 in /tests/e2e #1044 (2026-05-08).
  • Bumps node-version from '20' to '22' in .github/workflows/e2e-tests.yml to restore CI.

Test plan

  • E2E Tests check on this PR turns green.
  • No regressions in other CI checks (PHPCS, PHPUnit) — they don't touch Node.

The latest pnpm (11.1.3) requires Node.js v22.13+ and imports `node:sqlite`,
a built-in module that doesn't exist in Node 20. `actions/setup-node@v4`
crashes when setting up the pnpm cache, breaking the E2E job on every run
since #1044.

Bumping to Node 22 restores compatibility with the pnpm version pulled in by
`pnpm/action-setup@v4` (version: latest).
@donnchawp donnchawp merged commit dfaa0ad into trunk May 19, 2026
5 of 6 checks passed
@donnchawp donnchawp mentioned this pull request May 19, 2026
4 tasks
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