Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DRAFT: Let's see if CI passes on Node.js v23 #993

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Mar 13, 2025

@agracio @kkoopa Your review, please.

@agracio
Copy link

agracio commented Mar 13, 2025

I am not a member of nan team
I only participate in topics that are related to Electron builds 😄

@cclauss cclauss mentioned this pull request Mar 13, 2025
@agracio
Copy link

agracio commented Mar 13, 2025

Personally I use nan to build Node.js and node-electron native modules for my projects. Building Node.js v23 native module works perfectly fine but that does not mean that there no underlying issues just like the failing tests are showing.

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