Update supported Node versions #1199
Conversation
.github/workflows/test.yml
Outdated
| - uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 20 | ||
| node-version: 24 |
There was a problem hiding this comment.
I prefer using the oldest supported Node.js version in CI. It’s easy to verify compatibility with the latest version, but I never test with the oldest one locally. This approach ensures the version we claim to support actually works.
There was a problem hiding this comment.
Makes sense. Would switching to matrix testing, ensuring testing on the highest and lowest boundaries fulfill that want? 🙂
|
If at some point we want to speed up build time, I wouldn't be shocked if we only tested on macOS and Windows on Node-latest, and used only UNIX for Node range testing. The risk of an incompatibility existing on a specific platform + specific Node version combination seems low enough to me to justify sparing resources. |
Done in #1189, missed metadata update