endojs / endo

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

drop matrix from the lint job #2283

Open turadg opened 1 month ago

turadg commented 1 month ago

What is the Problem Being Solved?

The lint job does static analysis, but it's configured to run against multiple Node runtimes. Only one is needed and it shouldn't have a matrix because that necessitates updating its Required check when bumping Node versions.

Description of the Design

  1. Take the matrix out of https://github.com/endojs/endo/blob/b02f0e278c9fb19515cbab58635e6dbead2eb9bc/.github/workflows/ci.yml#L18-L28

  2. A repo admin updates settings to require the plain "lint" job, not the runtime-scoped one

Security Considerations

Scaling Considerations

Test Plan

Compatibility Considerations

Upgrade Considerations