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

[v13.x backport] stream: eos make const state const by ronag · Pull Request #32178 · nodejs/node · GitHub

/ node Public

[v13.x backport] stream: eos make const state const - #32178

Closed
ronag wants to merge 1 commit into
nodejs:v13.x-stagingfrom
nxtedition:backport-32031-to-v13.x
Closed

[v13.x backport] stream: eos make const state const#32178
ronag wants to merge 1 commit into
nodejs:v13.x-stagingfrom
nxtedition:backport-32031-to-v13.x

Conversation

ronag commented Mar 10, 2020
edited
Loading

Copy link
Copy Markdown
Member

PR-URL: #32031

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

writable & readable is based on type and is not actual
state, treat them as such.

PR-URL: nodejs#32031
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Backport-PR-URL: nodejs#32178
ronag force-pushed the backport-32031-to-v13.x branch from aa6df82 to 18e5e53 Compare March 10, 2020 17:32
ronag added the stream Issues and PRs related to the stream subsystem. label Mar 10, 2020

Copy link
Copy Markdown
Collaborator

MylesBorins pushed a commit that referenced this pull request Mar 10, 2020
writable & readable is based on type and is not actual
state, treat them as such.

Backport-PR-URL: #32178
PR-URL: #32031
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

Copy link
Copy Markdown
Contributor

landed in f263659

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

stream Issues and PRs related to the stream subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL