typings: improve typings by targos · Pull Request #40222 · nodejs/node

targos · 2021-09-26T14:36:03Z

typings: fix declaration of primordials
Closes: https://github.com/nodejs/node/issues/40144

typings: define types for timers binding

typings: add missing types to options and util bindings

typings: define types for os binding

Closes: nodejs#40144

Mesteery · 2021-09-26T16:18:49Z

typings/internalBinding/os.d.ts

+  getUptime(): number;
+  getTotalMem(): number;
+  getFreeMem(): number;
+  getCPUs(): Array<string | number>;


Suggested change

getCPUs(): Array<string | number>;

getCPUs(): (string | number)[];

for consistency with other dts

I have a preference to use type[] for simple types and Array<types> for more complex ones

Mesteery · 2021-09-26T16:19:08Z

typings/internalBinding/os.d.ts

+  getTotalMem(): number;
+  getFreeMem(): number;
+  getCPUs(): Array<string | number>;
+  getInterfaceAddresses(ctx: object): Array<string | number | boolean> | undefined;


typings/internalBinding/os.d.ts

mhdawson

LGTM

github-actions · 2021-10-09T17:11:44Z

Commit Queue failed

- Loading data for nodejs/node/pull/40222 ✔ Done loading data for nodejs/node/pull/40222 ----------------------------------- PR info ------------------------------------ Title typings: improve typings (#40222) Author Michaël Zasso (@targos) Branch targos:typings -> nodejs:master Labels author ready, typings Commits 5 - typings: fix declaration of primordials - typings: define types for timers binding - typings: add missing types to options and util bindings - typings: define types for os binding - fixup! typings: define types for os binding Committers 1 - Michaël Zasso PR-URL: https://github.com/nodejs/node/pull/40222 Fixes: https://github.com/https://github.com/nodejs/node/issues/40144 Reviewed-By: Tobias Nießen Reviewed-By: Evan Lucas Reviewed-By: Zijian Liu Reviewed-By: James M Snell Reviewed-By: Michael Dawson ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/40222 Fixes: https://github.com/https://github.com/nodejs/node/issues/40144 Reviewed-By: Tobias Nießen Reviewed-By: Evan Lucas Reviewed-By: Zijian Liu Reviewed-By: James M Snell Reviewed-By: Michael Dawson -------------------------------------------------------------------------------- ℹ This PR was created on Sun, 26 Sep 2021 14:36:03 GMT ✔ Approvals: 5 ✔ - Tobias Nießen (@tniessen) (TSC): https://github.com/nodejs/node/pull/40222#pullrequestreview-763706951 ✔ - Evan Lucas (@evanlucas): https://github.com/nodejs/node/pull/40222#pullrequestreview-764570178 ✔ - Zijian Liu (@Lxxyx): https://github.com/nodejs/node/pull/40222#pullrequestreview-766148910 ✔ - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/40222#pullrequestreview-767943198 ✔ - Michael Dawson (@mhdawson) (TSC): https://github.com/nodejs/node/pull/40222#pullrequestreview-769403509 ✔ Last GitHub Actions successful ℹ Green GitHub Actions CI is sufficient -------------------------------------------------------------------------------- ✔ No git cherry-pick in progress ✔ No git am in progress ✔ No git rebase in progress -------------------------------------------------------------------------------- - Bringing origin/master up to date... From https://github.com/nodejs/node * branch master -> FETCH_HEAD ✔ origin/master is now up-to-date - Downloading patch for 40222 From https://github.com/nodejs/node * branch refs/pull/40222/merge -> FETCH_HEAD ✔ Fetched commits as eaa59571e0fd..795db39025ef -------------------------------------------------------------------------------- [master 0b4ed8fe7f] typings: fix declaration of primordials Author: Michaël Zasso Date: Sun Sep 26 12:22:11 2021 +0200 1 file changed, 5 insertions(+), 1 deletion(-) [master c5028c4cfb] typings: define types for timers binding Author: Michaël Zasso Date: Sun Sep 26 15:47:14 2021 +0200 2 files changed, 9 insertions(+) create mode 100644 typings/internalBinding/timers.d.ts [master 7b025d4363] typings: add missing types to options and util bindings Author: Michaël Zasso Date: Sun Sep 26 16:07:13 2021 +0200 2 files changed, 2 insertions(+) [master 03be0fbe11] typings: define types for os binding Author: Michaël Zasso Date: Sun Sep 26 16:28:40 2021 +0200 2 files changed, 22 insertions(+) create mode 100644 typings/internalBinding/os.d.ts [master 88e9b9da17] fixup! typings: define types for os binding Author: Michaël Zasso Date: Mon Sep 27 16:50:49 2021 +0200 1 file changed, 7 insertions(+), 7 deletions(-) ✔ Patches applied There are 5 commits in the PR. Attempting autorebase. Rebasing (2/9)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
typings: fix declaration of primordials

Closes: #40144

PR-URL: #40222
Fixes: #40144
Reviewed-By: Tobias Nießen tniessen@tnie.de
Reviewed-By: Evan Lucas evanlucas@me.com
Reviewed-By: Zijian Liu lxxyxzj@gmail.com
Reviewed-By: James M Snell jasnell@gmail.com
Reviewed-By: Michael Dawson midawson@redhat.com

[detached HEAD 410ea3f9af] typings: fix declaration of primordials
Author: Michaël Zasso targos@protonmail.com
Date: Sun Sep 26 12:22:11 2021 +0200
1 file changed, 5 insertions(+), 1 deletion(-)
Rebasing (3/9)
Rebasing (4/9)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
typings: define types for timers binding

PR-URL: #40222
Fixes: #40144
Reviewed-By: Tobias Nießen tniessen@tnie.de
Reviewed-By: Evan Lucas evanlucas@me.com
Reviewed-By: Zijian Liu lxxyxzj@gmail.com
Reviewed-By: James M Snell jasnell@gmail.com
Reviewed-By: Michael Dawson midawson@redhat.com

[detached HEAD 71c9083927] typings: define types for timers binding
Author: Michaël Zasso targos@protonmail.com
Date: Sun Sep 26 15:47:14 2021 +0200
2 files changed, 9 insertions(+)
create mode 100644 typings/internalBinding/timers.d.ts
Rebasing (5/9)
Rebasing (6/9)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
typings: add missing types to options and util bindings

PR-URL: #40222
Fixes: #40144
Reviewed-By: Tobias Nießen tniessen@tnie.de
Reviewed-By: Evan Lucas evanlucas@me.com
Reviewed-By: Zijian Liu lxxyxzj@gmail.com
Reviewed-By: James M Snell jasnell@gmail.com
Reviewed-By: Michael Dawson midawson@redhat.com

[detached HEAD 56184dad50] typings: add missing types to options and util bindings
Author: Michaël Zasso targos@protonmail.com
Date: Sun Sep 26 16:07:13 2021 +0200
2 files changed, 2 insertions(+)
Rebasing (7/9)
Rebasing (8/9)
Rebasing (9/9)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
typings: define types for os binding

PR-URL: #40222
Fixes: #40144
Reviewed-By: Tobias Nießen tniessen@tnie.de
Reviewed-By: Evan Lucas evanlucas@me.com
Reviewed-By: Zijian Liu lxxyxzj@gmail.com
Reviewed-By: James M Snell jasnell@gmail.com
Reviewed-By: Michael Dawson midawson@redhat.com

[detached HEAD 2286b9fc7a] typings: define types for os binding
Author: Michaël Zasso targos@protonmail.com
Date: Sun Sep 26 16:28:40 2021 +0200
2 files changed, 22 insertions(+)
create mode 100644 typings/internalBinding/os.d.ts

Successfully rebased and updated refs/heads/master.
✖ 410ea3f9af075b40679d532d808a1a7d249e0455
✖ 4:7 Fixes must be a GitHub URL. fixes-url
✔ 0:0 blank line after title line-after-title
✔ 0:0 line-lengths are valid line-length
✔ 0:0 metadata is at end of message metadata-end
✔ 3:8 PR-URL is valid. pr-url
✔ 0:0 reviewers are valid reviewers
✔ 0:0 valid subsystems subsystem
✔ 0:0 Title is formatted correctly. title-format
✔ 0:0 Title is <= 50 columns. title-length
✖ 71c9083927b2c4f8c4a15c4bcd306d864bf77c65
✖ 2:7 Fixes must be a GitHub URL. fixes-url
✔ 0:0 blank line after title line-after-title
✔ 0:0 line-lengths are valid line-length
✔ 0:0 metadata is at end of message metadata-end
✔ 1:8 PR-URL is valid. pr-url
✔ 0:0 reviewers are valid reviewers
✔ 0:0 valid subsystems subsystem
✔ 0:0 Title is formatted correctly. title-format
✔ 0:0 Title is <= 50 columns. title-length
✖ 56184dad5073aa40ebb52a9faea80e33e852903b
✖ 2:7 Fixes must be a GitHub URL. fixes-url
✔ 0:0 blank line after title line-after-title
✔ 0:0 line-lengths are valid line-length
✔ 0:0 metadata is at end of message metadata-end
✔ 1:8 PR-URL is valid. pr-url
✔ 0:0 reviewers are valid reviewers
✔ 0:0 valid subsystems subsystem
✔ 0:0 Title is formatted correctly. title-format
⚠ 0:50 Title should be <= 50 columns. title-length
✖ 2286b9fc7a5cc65aa97b02a895c335e1937f5172
✖ 2:7 Fixes must be a GitHub URL. fixes-url
✔ 0:0 blank line after title line-after-title
✔ 0:0 line-lengths are valid line-length
✔ 0:0 metadata is at end of message metadata-end
✔ 1:8 PR-URL is valid. pr-url
✔ 0:0 reviewers are valid reviewers
✔ 0:0 valid subsystems subsystem
✔ 0:0 Title is formatted correctly. title-format
✔ 0:0 Title is <= 50 columns. title-length
ℹ Please fix the commit message and try again.

https://github.com/nodejs/node/actions/runs/1323958359

targos · 2021-10-10T09:04:28Z

Landed in 8068f40...9467cba

PR-URL: #40222 Fixes: #40144 Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: Zijian Liu <lxxyxzj@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <midawson@redhat.com>

PR-URL: #40222 Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: Zijian Liu <lxxyxzj@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <midawson@redhat.com>

PR-URL: #40222 Fixes: #40144 Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: Zijian Liu <lxxyxzj@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <midawson@redhat.com>

PR-URL: #40222 Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: Zijian Liu <lxxyxzj@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <midawson@redhat.com>

targos added 4 commits September 26, 2021 12:22

typings: fix declaration of primordials

7764c93

Closes: nodejs#40144

typings: define types for timers binding

6cfc899

typings: add missing types to options and util bindings

a7173b4

typings: define types for os binding

c2460eb

targos added the typings label Sep 26, 2021

tniessen approved these changes Sep 26, 2021

View reviewed changes

Mesteery reviewed Sep 26, 2021

View reviewed changes

fixup! typings: define types for os binding

795db39

evanlucas approved these changes Sep 27, 2021

View reviewed changes

Lxxyx approved these changes Sep 29, 2021

View reviewed changes

jasnell approved these changes Sep 30, 2021

View reviewed changes

VoltrexKeyva added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Sep 30, 2021

mhdawson approved these changes Oct 1, 2021

View reviewed changes

aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 9, 2021

github-actions bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Oct 9, 2021

targos removed the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Oct 10, 2021

targos closed this Oct 10, 2021

targos deleted the typings branch October 10, 2021 09:04

BethGriggs mentioned this pull request Oct 14, 2021

2021-10-19, Version 17.0.0 (Current) #40119

Merged

2 tasks

richardlau mentioned this pull request Oct 18, 2021

v16.12.0 proposal #40504

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

typings: improve typings#40222

typings: improve typings#40222
targos wants to merge 5 commits intonodejs:masterfrom
targos:typings

targos commented Sep 26, 2021

Uh oh!

Mesteery Sep 26, 2021

Uh oh!

targos Sep 27, 2021

Uh oh!

Mesteery Sep 26, 2021

Uh oh!

Uh oh!

Uh oh!

mhdawson left a comment

Uh oh!

github-actions bot commented Oct 9, 2021

PR-URL: #40222
Fixes: #40144
Reviewed-By: Tobias Nießen tniessen@tnie.de
Reviewed-By: Evan Lucas evanlucas@me.com
Reviewed-By: Zijian Liu lxxyxzj@gmail.com
Reviewed-By: James M Snell jasnell@gmail.com
Reviewed-By: Michael Dawson midawson@redhat.com

PR-URL: #40222
Fixes: #40144
Reviewed-By: Tobias Nießen tniessen@tnie.de
Reviewed-By: Evan Lucas evanlucas@me.com
Reviewed-By: Zijian Liu lxxyxzj@gmail.com
Reviewed-By: James M Snell jasnell@gmail.com
Reviewed-By: Michael Dawson midawson@redhat.com

PR-URL: #40222
Fixes: #40144
Reviewed-By: Tobias Nießen tniessen@tnie.de
Reviewed-By: Evan Lucas evanlucas@me.com
Reviewed-By: Zijian Liu lxxyxzj@gmail.com
Reviewed-By: James M Snell jasnell@gmail.com
Reviewed-By: Michael Dawson midawson@redhat.com

PR-URL: #40222
Fixes: #40144
Reviewed-By: Tobias Nießen tniessen@tnie.de
Reviewed-By: Evan Lucas evanlucas@me.com
Reviewed-By: Zijian Liu lxxyxzj@gmail.com
Reviewed-By: James M Snell jasnell@gmail.com
Reviewed-By: Michael Dawson midawson@redhat.com

Uh oh!

targos commented Oct 10, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

	getCPUs(): Array<string \| number>;
	getCPUs(): (string \| number)[];

	getInterfaceAddresses(ctx: object): Array<string \| number \| boolean> \| undefined;
	getInterfaceAddresses(ctx: object): (string \| number \| boolean)[] \| undefined;

Uh oh!

Conversation

targos commented Sep 26, 2021

Uh oh!

Mesteery Sep 26, 2021

Choose a reason for hiding this comment

Uh oh!

targos Sep 27, 2021

Choose a reason for hiding this comment

Uh oh!

Mesteery Sep 26, 2021

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

mhdawson left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Oct 9, 2021

PR-URL: #40222 Fixes: #40144 Reviewed-By: Tobias Nießen tniessen@tnie.de Reviewed-By: Evan Lucas evanlucas@me.com Reviewed-By: Zijian Liu lxxyxzj@gmail.com Reviewed-By: James M Snell jasnell@gmail.com Reviewed-By: Michael Dawson midawson@redhat.com

PR-URL: #40222 Fixes: #40144 Reviewed-By: Tobias Nießen tniessen@tnie.de Reviewed-By: Evan Lucas evanlucas@me.com Reviewed-By: Zijian Liu lxxyxzj@gmail.com Reviewed-By: James M Snell jasnell@gmail.com Reviewed-By: Michael Dawson midawson@redhat.com

PR-URL: #40222 Fixes: #40144 Reviewed-By: Tobias Nießen tniessen@tnie.de Reviewed-By: Evan Lucas evanlucas@me.com Reviewed-By: Zijian Liu lxxyxzj@gmail.com Reviewed-By: James M Snell jasnell@gmail.com Reviewed-By: Michael Dawson midawson@redhat.com

PR-URL: #40222 Fixes: #40144 Reviewed-By: Tobias Nießen tniessen@tnie.de Reviewed-By: Evan Lucas evanlucas@me.com Reviewed-By: Zijian Liu lxxyxzj@gmail.com Reviewed-By: James M Snell jasnell@gmail.com Reviewed-By: Michael Dawson midawson@redhat.com

Uh oh!

targos commented Oct 10, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants

PR-URL: #40222
Fixes: #40144
Reviewed-By: Tobias Nießen tniessen@tnie.de
Reviewed-By: Evan Lucas evanlucas@me.com
Reviewed-By: Zijian Liu lxxyxzj@gmail.com
Reviewed-By: James M Snell jasnell@gmail.com
Reviewed-By: Michael Dawson midawson@redhat.com

PR-URL: #40222
Fixes: #40144
Reviewed-By: Tobias Nießen tniessen@tnie.de
Reviewed-By: Evan Lucas evanlucas@me.com
Reviewed-By: Zijian Liu lxxyxzj@gmail.com
Reviewed-By: James M Snell jasnell@gmail.com
Reviewed-By: Michael Dawson midawson@redhat.com

PR-URL: #40222
Fixes: #40144
Reviewed-By: Tobias Nießen tniessen@tnie.de
Reviewed-By: Evan Lucas evanlucas@me.com
Reviewed-By: Zijian Liu lxxyxzj@gmail.com
Reviewed-By: James M Snell jasnell@gmail.com
Reviewed-By: Michael Dawson midawson@redhat.com

PR-URL: #40222
Fixes: #40144
Reviewed-By: Tobias Nießen tniessen@tnie.de
Reviewed-By: Evan Lucas evanlucas@me.com
Reviewed-By: Zijian Liu lxxyxzj@gmail.com
Reviewed-By: James M Snell jasnell@gmail.com
Reviewed-By: Michael Dawson midawson@redhat.com