Skip to content

doc: fix added: info for stream._construct() - #36067

Closed
lpinca wants to merge 1 commit into
nodejs:masterfrom
lpinca:fix/added-version-for-construct
Closed

lpinca wants to merge 1 commit into
nodejs:masterfrom
lpinca:fix/added-version-for-construct

Conversation

@lpinca

@lpinca lpinca commented Nov 10, 2020

Copy link
Copy Markdown
Member

readable._construct() and writable._construct() were added to
Node.js v15.0.0 via fb8cc72.

Fixes: #36058

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to Node.js documentation. stream Issues and PRs related to Node.js streams. labels Nov 10, 2020
`readable._construct()` and `writable._construct()` were added to
Node.js v15.0.0 via fb8cc72.

Fixes: nodejs#36058
@lpinca
lpinca force-pushed the fix/added-version-for-construct branch from 9da8acf to fa94436 Compare November 10, 2020 14:52
@lpinca lpinca changed the title doc: fix added: info for stream._construct() doc: fix added: info for stream._construct() Nov 10, 2020

@a0viedo a0viedo left a comment

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.

LGTM

@aduh95

aduh95 commented Nov 10, 2020

Copy link
Copy Markdown
Contributor

fast-track?

@aduh95 aduh95 added the fast-track PRs proposed for a shorter-than-standard waiting period before landing. label Nov 10, 2020
@aduh95
aduh95 requested a review from ronag November 10, 2020 17:31
@aduh95 aduh95 added the commit-queue PRs queued for automated landing through the Commit Queue. label Nov 10, 2020
@github-actions github-actions Bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Nov 10, 2020
@github-actions

Copy link
Copy Markdown
Contributor

Landed in 60a97c0...0dee498

@github-actions github-actions Bot closed this Nov 10, 2020
nodejs-github-bot pushed a commit that referenced this pull request Nov 10, 2020
`readable._construct()` and `writable._construct()` were added to
Node.js v15.0.0 via fb8cc72.

Fixes: #36058

PR-URL: #36067
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
@lpinca
lpinca deleted the fix/added-version-for-construct branch November 10, 2020 20:16
@targos

targos commented Nov 14, 2020

Copy link
Copy Markdown
Member

Pranay180420 pushed a commit to Pranay180420/Node.jsforme that referenced this pull request Feb 17, 2025
`readable._construct()` and `writable._construct()` were added to
Node.js v15.0.0 via fb8cc72.

Fixes: nodejs/node#36058

PR-URL: nodejs/node#36067
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to Node.js documentation. fast-track PRs proposed for a shorter-than-standard waiting period before landing. stream Issues and PRs related to Node.js streams.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Streams: readable stream does not get _construct called

7 participants