Skip to content

Conversation

@kzar
Copy link
Contributor

@kzar kzar commented Apr 3, 2020

Please also see the corresponding issue for more background: #32638

Checklist

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. stream Issues and PRs related to the stream subsystem. labels Apr 3, 2020
@kzar
Copy link
Contributor Author

kzar commented Apr 3, 2020

I've got 10.17.0 by looking at the changelogs, but I have only personally tested it's present in 10.19.0.

Copy link
Member

Choose a reason for hiding this comment

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

@nodejs/documentation What's the correct metadata here? Can we have multiple added: entries (for each release line)? This API never went into a Node.js 11.x release.

Copy link
Member

Choose a reason for hiding this comment

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

Yes. We should have multiple added: entries:

Suggested change
added: v10.17.0
added:
- v12.3.0
- v10.17.0

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, Done.

@kzar kzar force-pushed the 32638-readable-from-version branch from 5ded467 to d82758f Compare April 3, 2020 15:09
@addaleax addaleax added the fast-track PRs that do not need to wait for 48 hours to land. label Apr 3, 2020
@addaleax
Copy link
Member

addaleax commented Apr 3, 2020

Feel free to 👍 this comment to approve fast-tracking

@addaleax addaleax mentioned this pull request Apr 3, 2020
@addaleax
Copy link
Member

addaleax commented Apr 5, 2020

Landed in 8c0a17a

addaleax pushed a commit that referenced this pull request Apr 5, 2020
PR-URL: #32639
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@addaleax addaleax closed this Apr 5, 2020
BethGriggs pushed a commit that referenced this pull request Apr 6, 2020
PR-URL: #32639
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
BethGriggs pushed a commit that referenced this pull request Apr 7, 2020
PR-URL: #32639
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
targos pushed a commit that referenced this pull request Apr 12, 2020
PR-URL: #32639
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
targos pushed a commit that referenced this pull request Apr 22, 2020
PR-URL: #32639
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
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 the documentations. fast-track PRs that do not need to wait for 48 hours to land. stream Issues and PRs related to the stream subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants