endojs / endo

Endo is a distributed secure JavaScript sandbox, based on SES
Apache License 2.0
761 stars 68 forks source link

test: ci tests on node 22 rather than 21 #2322

Closed erights closed 2 weeks ago

erights commented 2 weeks ago

Most PRs should close a specific Issue. All PRs should at least reference one or more Issues. Edit and/or delete the following lines as appropriate (note: you don't need both refs and closes for the same one):

Closes: #2314 Refs: https://github.com/endojs/endo/pull/2231 https://github.com/endojs/endo/commit/b11bebae29f97e934c27dbb1d5f99cfa5d0a7627

Description

Now that nvm recognizes Node 22, we should probably redo the change from #2231 undone by https://github.com/endojs/endo/commit/b11bebae29f97e934c27dbb1d5f99cfa5d0a7627 .

Security Considerations

Whether our code works on Node 22 is much more relevant, including for security, than whether it works on Node 21. Indeed, until 22 could be used we were using Node 21 only for early warning about Node 22.

Scaling Considerations

none

Documentation Considerations

none

Testing Considerations

the point. Now that Node 22 is out, we should test on it.

Compatibility Considerations

Hopefully none

Upgrade Considerations

None