Skip to content

ci: drop node 20 from the full test matrix - #111

Merged
dprevost-LMI merged 1 commit into
webdriverio:mainfrom
dprevost-LMI:drop-node-20
Sep 22, 2026
Merged

dprevost-LMI merged 1 commit into
webdriverio:mainfrom
dprevost-LMI:drop-node-20

Conversation

@dprevost-LMI

Copy link
Copy Markdown
Collaborator

package.json's engines.node has required >=22.0.0 for a while, and .nvmrc/.cursor/Dockerfile already pin 24 - test.yml's push-to-main matrix was the only place still testing a version the package manifest already declares unsupported.

package.json's engines.node has required >=22.0.0 for a while, and
.nvmrc/.cursor/Dockerfile already pin 24 - test.yml's push-to-main matrix
was the only place still testing a version the package manifest already
declares unsupported.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@greptile-apps

greptile-apps Bot commented Sep 22, 2026

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

The PR appears safe to merge because the updated CI matrix continues to cover the repository’s declared supported Node.js versions.

Summary

This PR aligns the full CI test matrix with the package’s declared Node.js support range.

  • Removes unsupported Node.js 20 from push-to-main testing.
  • Retains coverage for Node.js 22, 24, and 26.
  • Leaves pull-request testing on the repository’s Node.js 24 target.

Reviews (1) · Last reviewed commit: "ci: drop node 20 from the full test matr..."

@dprevost-LMI
dprevost-LMI merged commit 39f7ec6 into webdriverio:main Sep 22, 2026
4 checks passed
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