nksaraf / vinxi

The Full Stack JavaScript SDK
https://vinxi.vercel.app
MIT License
2.19k stars 80 forks source link

Expand CI test matrix from Node 18 to 18/20/22 #413

Open birkskyum opened 2 weeks ago

birkskyum commented 2 weeks ago

The CI currently only check Node 18:

https://github.com/nksaraf/vinxi/blob/b0ccb64d3c37488050eb9411be4290ea466c3eba/.github/workflows/test.yml#L30

The active and current LTS version of Node is Node 20/22.

I'm sure a lot of people, myself included, use these versions.

codebryo commented 2 weeks ago

I fully agree. Would even argue it would make sense for a project like Vinxi to drop official 18 support.

While Vite states 18 and 20+ are supported, the v18 support is getting less in the eco system.

Nitro officially only states latest LTS, so I guess that while 22 is the latest LTS, 20 will be supported for a longer period as well.