Skip to content

Node.js 26 support#133

Merged
brianc merged 2 commits intobrianc:masterfrom
mureinik:nodejs-26-support
May 7, 2026
Merged

Node.js 26 support#133
brianc merged 2 commits intobrianc:masterfrom
mureinik:nodejs-26-support

Conversation

@mureinik
Copy link
Copy Markdown
Contributor

@mureinik mureinik commented May 6, 2026

Fix support for Node.js 26

This PR includes:

  • Bumping nan to 2.26.2 to support Node.js 26
  • Adding Node.js 26 to the test matrix in the CI
  • lockfiles are updated accordingly

Note that yargs@17 (indirectly required by mocha) is incompatible with Node.js 26, so for that version the CI force-installs yargs@18. This is done in the CI instead of adding an overrides section in package.json since yargs@18 requires Node.js 20.19.0, which is incompatible with some of the Node.js version the CI is run on.

Fixes #132

@mureinik mureinik force-pushed the nodejs-26-support branch from b88c7bb to 21574cb Compare May 6, 2026 19:51
@mureinik mureinik changed the title Nodejs 26 support Node.js 26 support May 6, 2026
@brianc
Copy link
Copy Markdown
Owner

brianc commented May 7, 2026

Perfecto! TY! I will get a release for this out today.

@brianc
Copy link
Copy Markdown
Owner

brianc commented May 7, 2026

hmmm looks like the build for node@26.x is failing. You got time to take a look as to why? 🙏

Note that Node.js 26 is incompatible with yargs@17 transitively required
by mocha, so a specific step was added to upgrade it to yargs@18

Related to brianc#132
@mureinik mureinik force-pushed the nodejs-26-support branch from 21574cb to 95aecf4 Compare May 7, 2026 15:59
@mureinik
Copy link
Copy Markdown
Contributor Author

mureinik commented May 7, 2026

hmmm looks like the build for node@26.x is failing. You got time to take a look as to why? 🙏

Eap!
That was my stupid mistake, sorry about that.
Overriding yargs in the package.json for Node.js 26 allows the tests to run, but it needs to be done before running npm install/npm ci.

I've updated the PR, and it should be OK now, but it needs your approval to re-run the CI.

@mureinik
Copy link
Copy Markdown
Contributor Author

mureinik commented May 7, 2026

I've updated the PR, and it should be OK now, but it needs your approval to re-run the CI.

FWIW, CI passes now (and thanks for the re-trigger!)

@brianc
Copy link
Copy Markdown
Owner

brianc commented May 7, 2026

thank you for the help!

@brianc brianc merged commit 59a1c9a into brianc:master May 7, 2026
5 checks passed
@brianc
Copy link
Copy Markdown
Owner

brianc commented May 7, 2026

published!

@mureinik mureinik deleted the nodejs-26-support branch May 8, 2026 02:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Can't install with Node.js 26

2 participants