Esri / arcgis-rest-js

compact, modular JavaScript wrappers for the ArcGIS REST API
https://developers.arcgis.com/arcgis-rest-js/
Apache License 2.0
347 stars 119 forks source link

Update automated unit tests to run on Node 20 #1138

Closed gavinr-maps closed 6 months ago

gavinr-maps commented 10 months ago

Describe the problem

Node v20 is now the current active LTS version: https://github.com/nodejs/release#release-schedule

image

Describe the proposed solution

So we should probably add v20 to the list of versions that we automatically test on:

https://github.com/Esri/arcgis-rest-js/blob/9ec89a2113dd616519398cc217309930fed7c7bc/.github/workflows/pull-request-tests.yml#L21

Alternatives considered

No response

Additional Information

Questions:

patrickarlt commented 10 months ago

I think we should remove 14 and 16.

0scvr commented 10 months ago

I would like to work on this issue, please :)

patrickarlt commented 9 months ago

@0scvr all yours!

gavinr-maps commented 6 months ago

Updated in #1144 - thanks @0scvr and @patrickarlt!