alleyinteractive/alley-scripts

Add Support for node 20 and npm 10

Closed this issue · 3 comments

Description

Now that node 20 has gone LTS, and npm 10 is bundled with both node 18 and node 20, we should support both explicitly.

Use Case

As a developer, I want to be able to use these packages in a project that is using node 20 and/or npm 10.

Acceptance Criteria

  • Bump the version of node to 20 and npm to 10 in our tests
  • Modify package.json files to explicitly declare support for node 20 and npm 10
  • Cut new releases as necessary (e.g., block-editor-tools)

@kevinfodness the 'new releases as necessary' uses an example of block-editor-tools, however that specific project already looks to be happy with node 20/npm 10... was this just an example of a package or would a new release be necessary here and I'm overlooking something obvious?

The package.json for alley-scripts was updated to support node 20 and npm 10 by modifying the engines to support > node 16 and > npm 8, yes, but the parent package.json requires node < 19. CI is using node 18 for tests/build/etc.

We should update this package to use node 20 and npm 10 in CI tests and as part of build/release while allowing support for node 16-20 in engines.

This was resolved via #531. For block-editor-tools, this was included in 0.6.0. We also have testing against Node 16-20 in place.