typings: add signal_wrap internal binding types by leah-1ee · Pull Request #65229 · nodejs/node · GitHub
Skip to content

typings: add signal_wrap internal binding types - #65229

Merged
nodejs-github-bot merged 2 commits into
nodejs:mainfrom
leah-1ee:typings-signal-wrap
Aug 17, 2026
Merged

typings: add signal_wrap internal binding types#65229
nodejs-github-bot merged 2 commits into
nodejs:mainfrom
leah-1ee:typings-signal-wrap

Conversation

@leah-1ee

Copy link
Copy Markdown
Contributor

Add typings for the signal_wrap internal binding.

Changes:

  • Added typings/internalBinding/signal_wrap.d.ts
  • Added signal_wrap to InternalBindingMap
  • Typed Signal.start(signum), stop(), and the onsignal callback

Add a SignalWrapBinding declaration for internalBinding('signal_wrap')
and wire it into InternalBindingMap.

Signed-off-by: leah-1ee <selee3196@gmail.com>
Copilot AI lite review requested due to automatic review settings August 11, 2026 23:48

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@nodejs-github-bot nodejs-github-bot added the typings Issues and PRs related to internal TypeScript declarations. label Aug 11, 2026
Comment thread typings/internalBinding/signal_wrap.d.ts Outdated
Signed-off-by: leah-1ee <dltjddms1028@gmail.com>
@daeyeon daeyeon added author ready PRs with CI started, the required approvals, and no outstanding review comments. commit-queue PRs queued for automated landing through the Commit Queue. labels Aug 17, 2026
@nodejs-github-bot nodejs-github-bot added commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. and removed commit-queue PRs queued for automated landing through the Commit Queue. labels Aug 17, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@daeyeon daeyeon added commit-queue PRs queued for automated landing through the Commit Queue. commit-queue-squash PRs the Commit Queue should land as one squashed commit. and removed commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. labels Aug 17, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 0795d82 into nodejs:main Aug 17, 2026
34 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot nodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Aug 17, 2026
aduh95 pushed a commit that referenced this pull request Aug 25, 2026
Add a SignalWrapBinding declaration for internalBinding('signal_wrap')
and wire it into InternalBindingMap.

Signed-off-by: leah-1ee <selee3196@gmail.com>
PR-URL: #65229
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com>
aduh95 pushed a commit that referenced this pull request Aug 25, 2026
Add a SignalWrapBinding declaration for internalBinding('signal_wrap')
and wire it into InternalBindingMap.

Signed-off-by: leah-1ee <selee3196@gmail.com>
PR-URL: #65229
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com>
aduh95 pushed a commit that referenced this pull request Aug 27, 2026
Add a SignalWrapBinding declaration for internalBinding('signal_wrap')
and wire it into InternalBindingMap.

Signed-off-by: leah-1ee <selee3196@gmail.com>
PR-URL: #65229
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs with CI started, the required approvals, and no outstanding review comments. commit-queue-squash PRs the Commit Queue should land as one squashed commit. typings Issues and PRs related to internal TypeScript declarations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants