Skip to content

Target node v24#57

Open
rak-phillip wants to merge 2 commits intorancher:masterfrom
rak-phillip:task/node-24
Open

Target node v24#57
rak-phillip wants to merge 2 commits intorancher:masterfrom
rak-phillip:task/node-24

Conversation

@rak-phillip
Copy link
Copy Markdown
Member

Summary

This targets Node 24 in CI

Fixes #16758

Occurred changes and/or fixed issues

  • Target node 24 in CI

Technical notes summary

I left the node version as-is in package.json. I don't think there's any requirement to add further restrictions for this project, but we should ensure that this builds on active LTS versions of node.

Areas or cases that should be tested

  • CI should pass without errors

Relates work

Signed-off-by: Phillip Rak <rak.phillip@gmail.com>
@rak-phillip rak-phillip requested review from nwmac and richard-cox April 2, 2026 16:57
Copy link
Copy Markdown
Member

@richard-cox richard-cox left a comment

Choose a reason for hiding this comment

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

Would you be able to update package.json as well? if we're building with 24 we should dev with 24 as well.

Signed-off-by: Phillip Rak <rak.phillip@gmail.com>
@rak-phillip rak-phillip requested a review from richard-cox April 7, 2026 17:58
@rak-phillip
Copy link
Copy Markdown
Member Author

@richard-cox sure thing, this has been updated in 49a3861. This is ready for another review.

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