Skip to content

chore: drop Node 20 support, require Node 22+#7

Merged
pilat merged 1 commit intomainfrom
chore/drop-node20-support
Jan 15, 2026
Merged

chore: drop Node 20 support, require Node 22+#7
pilat merged 1 commit intomainfrom
chore/drop-node20-support

Conversation

@pilat
Copy link
Owner

@pilat pilat commented Jan 15, 2026

Node 20 LTS ends April 2026. Bump minimum to Node 22.

Summary by CodeRabbit

  • Chores
    • Updated minimum Node.js version requirement to 22.0.0 or higher
    • Updated CI/CD pipeline to test against Node.js versions 22 and 24
    • Updated dependency management configuration for type definitions

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 15, 2026

📝 Walkthrough

Walkthrough

The PR updates the project's minimum Node.js version requirement from 20 to 22 across configuration files, updates CI workflows to test against Node.js versions 22 and 24, and configures Dependabot to ignore major version updates for @types/node.

Changes

Cohort / File(s) Summary
Node.js Version Management
.github/dependabot.yml, package.json, .github/workflows/ci.yml
Synchronized Node.js version constraints: minimum engine requirement updated to >=22.0.0, CI matrix updated to test against versions 22 and 24 (from 20 and 22), and Dependabot configured to skip major version updates for @types/node

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Hopping forward to version twenty-two,
We've updated constraints through and through,
Node engines march, CI workflows align,
@types/node stays put—a design so fine! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: drop Node 20 support, require Node 22+' accurately summarizes the main changes: updating package.json engine requirement, CI workflow matrix, and dependabot configuration to drop Node 20 support and require Node 22+.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.



📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5e3f6d9 and 5bc6595.

📒 Files selected for processing (3)
  • .github/dependabot.yml
  • .github/workflows/ci.yml
  • package.json
🔇 Additional comments (3)
.github/dependabot.yml (1)

8-10: LGTM - sensible ignore rule for controlled Node version management.

Ignoring major @types/node updates prevents unwanted PRs for type definitions that don't match your supported Node versions. Just remember to manually update @types/node when you bump the minimum Node version in the future.

package.json (1)

42-44: LGTM - engine requirement correctly updated.

The engines.node field is properly updated to >=22.0.0, and @types/node at ^22.10.5 (line 59) is consistent with this requirement.

.github/workflows/ci.yml (1)

13-15: No action needed. Node.js 24 was already released on May 6, 2025, and entered Active LTS on October 28, 2025. The matrix configuration is appropriate and Node 24 is fully available in actions/setup-node.

Likely an incorrect or invalid review comment.

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.


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 and usage tips.

@pilat pilat merged commit 46ba2f8 into main Jan 15, 2026
3 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