Skip to content

CI: Add node 26, drop node 20 - #1245

Merged
MatthijsBurgh merged 1 commit into
developfrom
node_versions
Aug 3, 2026
Merged

CI: Add node 26, drop node 20#1245
MatthijsBurgh merged 1 commit into
developfrom
node_versions

Conversation

@MatthijsBurgh

Copy link
Copy Markdown
Contributor

Test against node 22, 24 and 26, bump the docs job to node 26 and raise the declared engines range to >=22.

Public API Changes

Description

Test against node 22, 24 and 26, bump the docs job to node 26 and raise
the declared engines range to >=22.

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

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.

Pull request overview

This PR updates the repository’s supported and tested Node.js versions by dropping Node 20 and adding Node 26, aligning CI, docs generation, and the packages/roslib engine requirement to a Node >=22 baseline.

Changes:

  • Update CI matrix to test Node.js 22, 24, and 26 (removing 20).
  • Bump docs workflow Node.js version to 26.
  • Raise packages/roslib declared Node.js engine requirement from >=20 to >=22.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
packages/roslib/package.json Raises the Node.js engine requirement to >=22 to match the new support baseline.
.github/workflows/main.yml Updates the CI test matrix to run on Node 22/24/26 instead of 20/22/24.
.github/workflows/docs.yml Bumps docs generation to run on Node 26.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@MatthijsBurgh
MatthijsBurgh merged commit c86ca6c into develop Aug 3, 2026
9 checks passed
@MatthijsBurgh
MatthijsBurgh deleted the node_versions branch August 3, 2026 07:48
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