endojs / endo

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

test(ci): Node.js 22 in CI #2231

Closed kriskowal closed 2 months ago

kriskowal commented 2 months ago

refs: #2230

Description

This change adds latest to the CI node setup action so we can see upcoming breaking changes to our platform. The corresponding test results will not be required for merge.

Testing canary did not pan out. You can get 22-v8-canary out of actions/setup-node, and you can add yarn install --ignore-engines, but yarn will still die with error if any dependency specifies engines since the canary versions do not match other version ranges, even broad ones like >= 16. This may be worth revisiting with newer versions of yarn or whatever succeeds it.

Security Considerations

This increases our visibility into breaking changes on the Node.js platform.

Scaling Considerations

This may prolong CI runs by a multiplicative factor if CI workers are contended.

Documentation Considerations

None.

Testing Considerations

Increases visibility into our future.

Compatibility Considerations

None.

Upgrade Considerations

None.

erights commented 2 months ago

The CI failures for ubuntu-latest look like the ones fixed by #2229, which is what's supposed to happen! Great!