Skip to content

typings: improve internal bindings typings - #40411

Closed
Mesteery wants to merge 1 commit into
nodejs:masterfrom
Mesteery:improve-bindings-typings
Closed

typings: improve internal bindings typings #40411
Mesteery wants to merge 1 commit into
nodejs:masterfrom
Mesteery:improve-bindings-typings

Conversation

@Mesteery

Copy link
Copy Markdown
Contributor

Depends on #40409.

Comment threadtypings/internalBinding/types.d.ts Outdated
Comment threadtypings/internalBinding/fs.d.ts Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

does it work now? Last time I tried, if checkJs was on, it introduced errors.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

It doesn't seem to me that there were any errors. It remains to be verified

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

I checked, there is no error with unique symbol.

@Mesteery
Mesteeryforce-pushed the improve-bindings-typings branch from 1bf68f3 to 805653eCompareOctober 24, 2021 17:07
@MesteeryMesteery added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Oct 24, 2021
@Ayase-252Ayase-252 added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 28, 2021
@github-actionsgithub-actionsBot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 28, 2021
@github-actions

Copy link
Copy Markdown
Contributor

Landed in 3ff8c35...b66a75a

nodejs-github-bot pushed a commit that referenced this pull request Oct 28, 2021
PR-URL: #40411
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit that referenced this pull request Nov 6, 2021
PR-URL: #40411
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@targostargos mentioned this pull request Nov 8, 2021
BethGriggs pushed a commit that referenced this pull request Nov 25, 2021
PR-URL: #40411
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@BethGriggsBethGriggs mentioned this pull request Nov 26, 2021
1 task
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.typings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@Mesteery@jasnell@targos@VoltrexKeyva@Ayase-252