Skip to content

ci: add node 26 to asan and valgrind matrices#321

Closed
jimexist wants to merge 1 commit into
DataDog:mainfrom
jimexist:claude/vibrant-bardeen-9f6cd4
Closed

ci: add node 26 to asan and valgrind matrices#321
jimexist wants to merge 1 commit into
DataDog:mainfrom
jimexist:claude/vibrant-bardeen-9f6cd4

Conversation

@jimexist
Copy link
Copy Markdown

@jimexist jimexist commented May 7, 2026

Summary

  • Add Node.js 26 to the asan and valgrind job matrices in .github/workflows/build.yml so CI exercises the latest Node release line.

Test plan

  • CI passes for the new Node 26 entry on both asan and valgrind jobs.

🤖 Generated with Claude Code

@IlyasShabi
Copy link
Copy Markdown

Thanks for this PR. Actually Node.js comes with V8 14.6 and some breaking changes. I already created two PRs to support Node.js on Nan and node-gyp:

@jimexist
Copy link
Copy Markdown
Author

thanks for the progress @IlyasShabi i wonder if now this PR is ready to run?

@jimexist jimexist force-pushed the claude/vibrant-bardeen-9f6cd4 branch 2 times, most recently from 511e4d2 to f3efdf2 Compare May 14, 2026 06:44
@IlyasShabi IlyasShabi mentioned this pull request May 18, 2026
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@jimexist jimexist force-pushed the claude/vibrant-bardeen-9f6cd4 branch from f3efdf2 to efb29a2 Compare May 18, 2026 13:37
@IlyasShabi
Copy link
Copy Markdown

I managed to have v26 supported in #327.
Thanks for your collaboration. Closing this PR

@IlyasShabi IlyasShabi closed this May 18, 2026
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.

2 participants