Skip to content

chore: Update node test versions#50

Merged
jamesnrokt merged 2 commits into
mainfrom
chore/update-node-test-versions
Jan 13, 2026
Merged

chore: Update node test versions#50
jamesnrokt merged 2 commits into
mainfrom
chore/update-node-test-versions

Conversation

@jamesnrokt

Copy link
Copy Markdown
Contributor

Background

  • Currently we can't update dependencies due to incompatibility of tests running using extremely old node version

What Has Changed

  • Updated the test matrix to run tests on newer versions of node

Screenshots/Video

  • N/A

Checklist

  • I have performed a self-review of my own code.

Additional Notes

  • N/A

Reference Issue (For employees only. Ignore if you are an outside contributor)

  • Closes N/A

@jamesnrokt jamesnrokt requested a review from a team as a code owner January 12, 2026 22:00
Copilot AI review requested due to automatic review settings January 12, 2026 22:00

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 Node.js test matrix in the CI workflow from very old versions (10.x, 12.x, 14.x, 15.x) to newer versions (19.x, 20.x, 21.x, 22.x) to enable dependency updates that are incompatible with the older Node versions.

Changes:

  • Updated the node-version matrix in the pull request workflow to test against Node.js 19.x, 20.x, 21.x, and 22.x instead of 10.x, 12.x, 14.x, and 15.x

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

Comment thread .github/workflows/pull-request.yml Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@jamesnrokt jamesnrokt merged commit cec38f6 into main Jan 13, 2026
10 checks passed
@jamesnrokt jamesnrokt deleted the chore/update-node-test-versions branch January 13, 2026 14:44
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.

3 participants