Skip to content

typings: wire up quic internal binding types - #65556

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
agape1225:typings-wire-up-quic-binding
Sep 21, 2026
Merged

nodejs-github-bot merged 1 commit into
nodejs:mainfrom
agape1225:typings-wire-up-quic-binding

Conversation

@agape1225

Copy link
Copy Markdown
Contributor

typings/internalBinding/quic.d.ts already declares QuicBinding, but it was never imported into globals.d.ts or added to InternalBindingMap, so internalBinding('quic') resolved to any instead of it.

@nodejs-github-bot nodejs-github-bot added the typings Issues and PRs related to internal TypeScript declarations. label Aug 26, 2026
typings/internalBinding/quic.d.ts already declares QuicBinding, but
it was never imported into globals.d.ts or added to
InternalBindingMap, so internalBinding('quic') resolved to `any`
instead of it.

Signed-off-by: agape1225 <49804691+agape1225@users.noreply.github.com>
@agape1225
agape1225 force-pushed the typings-wire-up-quic-binding branch from 1c093ff to a77966e Compare September 12, 2026 05:08
@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 Sep 21, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 35f72de into nodejs:main Sep 21, 2026
22 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 35f72de

@nodejs-github-bot nodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Sep 21, 2026
aduh95 pushed a commit that referenced this pull request Sep 27, 2026
typings/internalBinding/quic.d.ts already declares QuicBinding, but
it was never imported into globals.d.ts or added to
InternalBindingMap, so internalBinding('quic') resolved to `any`
instead of it.

Signed-off-by: agape1225 <49804691+agape1225@users.noreply.github.com>
PR-URL: #65556
Reviewed-By: James M Snell <jasnell@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. typings Issues and PRs related to internal TypeScript declarations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants