Skip to content

Conversation

@mfdebian
Copy link
Contributor

@mfdebian mfdebian commented Jan 6, 2026

This PR adds newer versions of Node to the test matrix to ensure compatibility ahead of the 5.0 release.

@mfdebian mfdebian mentioned this pull request Jan 6, 2026
9 tasks
Apply changes from code review

Co-authored-by: Sebastian Beltran <bjohansebas@gmail.com>
@mfdebian
Copy link
Contributor Author

mfdebian commented Jan 6, 2026

@bjohansebas sorry our conversation was marked as resolved when I committed your changes but as for adding a non LTS version of Node to the matrix, I think we can add the current release (v25)

would that be enough in your opinion? I think since we're adding tests for v22 and v24 there's no need to add one for v23? but I don't know, let me know what you think!

@bjohansebas
Copy link
Member

Well, it’s been our practice to have CI for all the versions we support, including non-LTS ones. In my opinion, it would be good to test all the supported LTS versions, to keep consistency with the other repositories

Copy link
Member

@bjohansebas bjohansebas left a comment

Choose a reason for hiding this comment

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

LGTM!

@mfdebian
Copy link
Contributor Author

mfdebian commented Jan 7, 2026

I've added non-LTS and the current Node release versions, lmkwyt!

@UlisesGascon UlisesGascon merged commit db39def into pillarjs:5.0 Jan 15, 2026
15 checks passed
@UlisesGascon UlisesGascon self-assigned this Jan 15, 2026
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.

4 participants