citgm icon indicating copy to clipboard operation
citgm copied to clipboard

Use pkgjs/action for CI to auto-upgrade to latest nodes

Open dominykas opened this issue 3 years ago • 3 comments

The CI workflow used here is almost identical to a standard Node.js library testing pattern (checkout, npm install, npm test).

As part of Package Maintenance WG, I've recently created pkgjs/action - it basically does that standard Node testing pattern, but it automatically maintains the test matrix for you, i.e. it will automatically detect the lowest supported (major) version from engines field and will automatically add new Node.js versions as they come out.

As the action is pretty new, I'd like to get some battle testing for it.

Would people mind if I opened a PR to start using it here? I'm willing to take on the maintenance of that until we feel it's stable.

The end result here would probably look something like this: https://github.com/nodejs/undici/blob/main/.github/workflows/nodejs.yml

dominykas avatar Jan 31 '22 15:01 dominykas

@nodejs/citgm

Trott avatar Jan 31 '22 23:01 Trott

I was going to open a PR to remove Node.js 12 from the CI workflow, but then realised this would be a preferable approach (IMO).

+1 to doing this.

BethGriggs avatar May 05 '22 15:05 BethGriggs

Opened a #907, although I'm thinking if it is enough of an improvement if the same is not applied to the test-module workflow... and I need to think about whether a feature to check out an arbitrary repo via pkgjs/action makes sense... Maybe?

dominykas avatar May 09 '22 17:05 dominykas