ci: add Node version matrix and expand target coverage#9
ci: add Node version matrix and expand target coverage#9antonymarion wants to merge 1 commit intoLadybugDB:mainfrom
Conversation
Add explicit Node 18/20/22 coverage to CI, add macOS x64 to match packaged targets, and include a non-blocking Node 24 canary job to catch upcoming runtime regressions early. Made-with: Cursor
adsharma
left a comment
There was a problem hiding this comment.
Thank you for working on this. We certainly want to expand the supported node version matrix, but do it in the right place while optimizing the CI cost/latency.
You could argue that this repo is the right place to build packages (and I agree), but it takes some work to make the move.
Yep did not check that the original ladybug repo (not the nodejs one) has the logic for build and deploy for extending the matrix compliance with several versions too. thks! |
Summary
18,20,22)actions/setup-nodeso CI no longer depends on runner-default Node versionsmacos-13(darwin-x64) to match packaged binary targets24canary job to surface upcoming runtime breakage earlyWhy
Recent Node runtime issues can slip through when CI does not pin/test multiple Node versions. This change makes runtime coverage explicit and improves confidence for ABI/runtime compatibility checks.
Test plan