FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

net: fix typo by lpinca · Pull Request #38127 · nodejs/node · GitHub

/ node Public

net: fix typo - #38127

Closed
lpinca wants to merge 1 commit into
nodejs:masterfrom
lpinca:fix/issue-38125
Closed

net: fix typo#38127
lpinca wants to merge 1 commit into
nodejs:masterfrom
lpinca:fix/issue-38125

Conversation

lpinca commented Apr 7, 2021
edited
Loading

Copy link
Copy Markdown
Member

SocketAddress.prototype.construtor -> SocketAddress.prototype.constructor.

Fixes: #38125

nodejs-github-bot added the needs-ci PRs that need a full CI run. label Apr 7, 2021
`SocketAddress.prototype.construtor` -> `SocketAddress.prototype.constructor`.

Fixes: nodejs#38125
lpinca force-pushed the fix/issue-38125 branch from 43ee622 to 7f7f135 Compare April 7, 2021 08:02

This comment has been minimized.

aduh95 added author ready PRs that have at least one approval, no outstanding review comments, and a CI started. fast-track PRs that do not need to wait for 48 hours to land. labels Apr 7, 2021

aduh95 commented Apr 7, 2021

Copy link
Copy Markdown
Contributor

fast-track?

This comment has been minimized.

This comment has been minimized.

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

Copy link
Copy Markdown
Collaborator

aduh95 pushed a commit that referenced this pull request Apr 8, 2021
`construtor` -> `constructor`.

Fixes: #38125

PR-URL: #38127
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>

aduh95 commented Apr 8, 2021

Copy link
Copy Markdown
Contributor

Landed in 7869761

aduh95 closed this Apr 8, 2021
lpinca deleted the fix/issue-38125 branch April 8, 2021 17:30
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no outstanding review comments, and a CI started. fast-track PRs that do not need to wait for 48 hours to land. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Typo in lib/internal/socketaddress.js

Back | FazBrowse Home | New Git URL