ThePalaceProject / web-patron / 30825305591

82%

push

github

Drop Node 20 support and add Node 22 support (#172)

## Description

<!--- Describe your changes -->
- Updates `engines` constraints in `package.json` to specify minimum
support for Node 22 (`node: >=22.0.0 <27.0.0`)
- Pins npm to 11.17.0 for Dockerfile. When not specified, the Dockerfile
uses default npm bundled with Node. For Node 20 and 22, this is npm 10.
- Swaps Node 22 for 20 in the CI matrix and specifies Coveralls coverage
report to use Node 22
- Updates `.nvmrc` to use Node 22.23.2 as the default version for
development

## Motivation and Context

<!--- Why is this change required? What problem does it solve? -->
<!--- If it fixes an open issue, please link to the issue here. -->
[PP-4708](https://ebce-lyrasis.atlassian.net/browse/PP-4708) introduces
translation support and relies on `i18next-cli` to manage and sync
translations during development. The latest version of `i18next-cli`
requires `node: >=22.0.0`.

As well, Node 20 is on End-Of-Life (EOL) and is thus no longer supported
with security updates. As time goes on, other packages will likely
continue to drop support for Node 20 as well, thus limiting access to
tools in the Node/npm ecosystem.

## How Has This Been Tested?

<!--- Please describe in detail how you tested your changes. -->
<!--- Include details of your testing environment, and the tests you ran
to -->
<!--- see how your change affects other areas of the code, etc. -->
- New CI matrix tests passes
- Local test suites pass

## Checklist:

<!--- Go over all the following points, and put an `x` in all the boxes
that apply. -->
<!--- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->

- [ ] I have updated the documentation accordingly.
- [x] All new and existing tests passed.


[PP-4708]:
https://ebce-lyrasis.atlassian.net/browse/PP-4708?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ

296 of 393 branches covered (75.32%)

Branch coverage included in aggregate %.

503 of 584 relevant lines covered (86.13%)

29.74 hits per line