Skip to content

Drop Node.js 16 from CI matrix and add Node.js 24#549

Merged
nev21 merged 3 commits intomainfrom
copilot/drop-nodejs-16-from-ci-matrix
Apr 23, 2026
Merged

Drop Node.js 16 from CI matrix and add Node.js 24#549
nev21 merged 3 commits intomainfrom
copilot/drop-nodejs-16-from-ci-matrix

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 23, 2026 •
edited
Loading

  • Locate actionable PR review comment
  • Update changelog entry to include a link to issue Drop Node.js 16 from CI matrix #548
  • Verify the changelog diff is limited to the requested line
  • Reply to the review comment with the addressing commit hash

Copilot AI linked an issue Apr 23, 2026 that may be closed by this pull request
Drop Node.js 16 from CI matrix #548
Closed
@nev21 nev21 marked this pull request as ready for review April 23, 2026 02:28
Copilot AI review requested due to automatic review settings April 23, 2026 02:28
Copilot AI reviewed Apr 23, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Copilot AI requested review from a team as code owners April 23, 2026 02:28
Copilot AI changed the title [WIP] Remove Node.js 16 from CI matrix due to end-of-life Drop Node.js 16 from CI matrix and add Node.js 24 Apr 23, 2026
Copilot AI requested a review from nev21 April 23, 2026 02:30
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 23, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.46%. Comparing base (1ac30dc) to head (97fbe40).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main     #549   +/-   ##
=======================================
  Coverage   99.46%   99.46%           
=======================================
  Files         145      145           
  Lines        4271     4271           
  Branches      938      938           
=======================================
  Hits         4248     4248           
  Misses         23       23
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

nev21
nev21 reviewed Apr 23, 2026
View reviewed changes
Comment thread CHANGELOG.md Outdated
@nev21 nev21 enabled auto-merge (squash) April 23, 2026 06:04
Copilot AI requested a review from nev21 April 23, 2026 06:05
nevware21-bot
nevware21-bot approved these changes Apr 23, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@nevware21-bot nevware21-bot left a comment

Choose a reason for hiding this comment

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

Approved by nevware21-bot

@nev21 nev21 merged commit 46c0418 into main Apr 23, 2026
10 checks passed
@nev21 nev21 deleted the copilot/drop-nodejs-16-from-ci-matrix branch April 23, 2026 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@nevware21-bot nevware21-bot nevware21-bot approved these changes

@nev21 nev21 Awaiting requested review from nev21

Assignees

@nev21 nev21

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Drop Node.js 16 from CI matrix

4 participants

@nev21 @nevware21-bot