codemod icon indicating copy to clipboard operation
codemod copied to clipboard

ci: update NodeJS versions tested

Open eventualbuddha opened this issue 6 months ago • 1 comments

Maintenance LTS, Active LTS, and Current.

eventualbuddha avatar Jul 11 '25 20:07 eventualbuddha

To elaborate on what I said about this PR in #927, CI is failing because of a new deprecation warning added to node 24 about the url.parse API (the warning gets printed to stderr, which a test checks is empty).

We could hack the test to disable checking stderr, but it would be much better just to not use the problematic URL parser. We're currently using cross-fetch/node-fetch, which is what is making the problematic call. Now that fetch is natively everywhere (including node), those libraries are unmaintained, so we need to switch to native fetch, not just update a library.

Like I said in #927, I did try to switch to native fetch without migrating to ESM, but I could not get that to work properly. The ESM migration will either fix it, or at least make the whole environment much easier to reason about.

ItsHarper avatar Aug 14 '25 15:08 ItsHarper