GitHub Actions: node: [16.x, 18.x, 20.x]

[cclauss]

Let's try to get brain.js tests passing on current versions of Node.js. https://github.com/nodejs/release#readme

• Had to disable macOS on Node.js v20 -- TODO: Fix this after this pull request lands.
• Had to install libxi-dev and friends on Ubuntu -- https://github.com/BrainJS/brain.js#installation-note
• Had to rm package-lock.json because it was using an out-of-date version of node-gyp
• Had to disable nine test files to get the tests to pass -- TODO: Fix this after this pull request lands.
  • Related to The Recurrent class will not function without a GPU #901 (comment) ?
• Had to fix up npm clean for Windows, etc.

brain.js/package-lock.json

Lines 11483 to 11484 in 2268193

	"node_modules/node-gyp": {
	"version": "7.1.2",

vs. v9.4.0 at https://github.com/nodejs/node-gyp/releases

Description

Motivation and Context

issue

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

Author's Checklist:

• My code focuses on the main motivation and avoids scope creep.
• My code passes current tests and adds new tests where possible.
• My code is SOLID and DRY.
• I have updated the documentation as needed.

Reviewer's Checklist:

• I kept my comments to the author positive, specific, and productive.
• I tested the code and didn't find any new problems.
• I think the motivation is good for the project.
• I think the code works to satisfy the motivation.

@robertleeplummerjr Your review, please.
@kul-sudo FYI

[robertleeplummerjr]

LGTM

[cclauss]

macOS failures are fixed by upgrading to the current https://github.com/nodejs/node-gyp/releases

[robertleeplummerjr]

Could you put in a PR that has this change?