fix: use Node.js 18

[brillout]

Using Node.js 18 instead of Node.js 16 should fix VPS's CI.

Also potentially the Laravel CI since its TS error is also about Node.js (CC @timacdonald).

Currently failing eco CI: vitejs/vite#11110 (comment)

[patak-dev]

This sounds good to me, feel free to merge @dominikg

[dominikg]

are you sure this fixed it? afaik node18 is a bit slower than node16 and the executing runtime should not affect the ts types? or is your CI running exclusively in node18?

i changed the types/node override to only apply to vite-3 tests, #155 but maybe it needs to be removed altogether instead?

[brillout]

Yes, in my case, that specific test is exclusively run with Node.js 18. This test uses res.writeEarlyHints() which was introduced in Node.js 18 so this PR should fix the problem (although I'm not sure whether #155 will interfere).

[timacdonald]

Thanks for the ping. I think the issue our package was hitting in the Vite CI pipeline was related to this fix: #154

We had to update these types in our package locally because they started conflicting with a recent TS version.