Skip to content

Removed Node 20 from CI matrix - #822

Merged
cmraible merged 1 commit into
mainfrom
chore/remove-node-20-ci
Aug 10, 2026
Merged

Removed Node 20 from CI matrix#822
cmraible merged 1 commit into
mainfrom
chore/remove-node-20-ci

Conversation

@cmraible

@cmraible cmraible commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

no refs

Summary

  • remove Node 20 from the CI test matrix
  • retain Node 22 and the Dockerfile-selected Node version

Why?

This repo is only published as a container image, so we control the node version in the public distribution. We only publish a Node 22 container, so we shouldn't need to run tests in Node 20 anymore.

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

The CI Node.js matrix now includes Node 22 and the extracted Dockerfile version. It no longer includes Node 20. Duplicate removal remains enabled.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly states the main change: removing Node 20 from the CI matrix.
Description check ✅ Passed The description directly explains the removal of Node 20 and the retained Node 22 and Dockerfile-selected versions.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/remove-node-20-ci

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cmraible
cmraible marked this pull request as ready for review August 6, 2026 15:27
@cmraible

cmraible commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator Author

Note: we'll need to remove the Node 20 ruleset from the branch protection rules before we can merge this.

@cmraible
cmraible requested a review from JoeeGrigg August 6, 2026 15:29
@cmraible cmraible changed the title Remove Node 20 from CI matrix Removed Node 20 from CI matrix Aug 6, 2026

@JoeeGrigg JoeeGrigg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems reasonable to me

@cmraible
cmraible merged commit 707b138 into main Aug 10, 2026
6 checks passed
@cmraible
cmraible deleted the chore/remove-node-20-ci branch August 10, 2026 14:20
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