Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v14.10.1 proposal #35137

Merged
merged 3 commits into from
Sep 10, 2020
Merged

v14.10.1 proposal #35137

merged 3 commits into from
Sep 10, 2020

Conversation

richardlau
Copy link
Member

@richardlau richardlau commented Sep 10, 2020

Notable Changes

Node.js 14.10.0 included a streams regression with async generators
and a docs rendering regression that are being fixed in this release.

Commits

  • [3c92f93b44] - doc: restore color for visited links (Rich Trott) #35108
  • [0f94c6b4e4] - Revert "stream: simpler and faster Readable async iterator" (Richard Lau)

@nodejs/streams @nodejs/tsc @nodejs/releasers
This proposes reverting 4bb4007 to address #35116. It sounds like more time is needed to otherwise fix the issues in both the streams code and in got and I think it would be better for us to get v14.x into a stable state ahead of Tuesday's security release.

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

@nodejs-github-bot nodejs-github-bot added lib / src Issues and PRs related to general changes in the lib or src directory. v14.x labels Sep 10, 2020
richardlau added a commit that referenced this pull request Sep 10, 2020
Notable changes:

Node.js 14.10.0 included a streams regression that is being fixed in
this release.

PR-URL: #35137
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Sep 10, 2020

Copy link
Contributor

@mmarchini mmarchini left a comment

Choose a reason for hiding this comment

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

Reverting the commit so we have more time to work on a long term solution sgtm

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Member

@Trott Trott left a comment

Choose a reason for hiding this comment

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

LGTM. Would love to get #35108 in this release as well.

@Trott Trott mentioned this pull request Sep 10, 2020
2 tasks
A previous change altered the rendering of visited links. This restores
it.

PR-URL: #35108
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Derek Lewis <DerekNonGeneric@inf.is>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Notable changes:

Node.js 14.10.0 included a streams regression with async generators
and a docs rendering regression that are being fixed in this release.

PR-URL: #35137
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Sep 10, 2020

@richardlau
Copy link
Member Author

LGTM. Would love to get #35108 in this release as well.

@Trott Added #35108.

richardlau added a commit that referenced this pull request Sep 10, 2020
@richardlau richardlau merged commit 4c2b513 into v14.x Sep 10, 2020
richardlau added a commit that referenced this pull request Sep 10, 2020
Notable changes:

Node.js 14.10.0 included a streams regression with async generators
and a docs rendering regression that are being fixed in this release.

PR-URL: #35137
richardlau added a commit to richardlau/nodejs.org that referenced this pull request Sep 10, 2020
richardlau added a commit to nodejs/nodejs.org that referenced this pull request Sep 10, 2020
@richardlau
Copy link
Member Author

richardlau commented Sep 10, 2020

There was a release process issue that seems to have resulted in the latest symlinks

not pointing to the new release. The actual files in https://nodejs.org/download/release/v14.10.1/ look okay.

FYI @ljharb in case this affects nvm

I've raised nodejs/build#2432.

@richardlau richardlau deleted the v14.10.1-proposal branch September 10, 2020 16:19
@ljharb
Copy link
Member

ljharb commented Sep 10, 2020

Luckily nvm doesn’t use those :-) thanks for the ping.

@Flarna
Copy link
Member

Flarna commented Sep 10, 2020

nvm for windows seems to be effected and the docs on website.

@richardlau
Copy link
Member Author

Symlinks have now been fixed.

joesepi pushed a commit to joesepi/node that referenced this pull request Jan 8, 2021
Notable changes:

Node.js 14.10.0 included a streams regression with async generators
and a docs rendering regression that are being fixed in this release.

PR-URL: nodejs#35137
@targos targos added release Issues and PRs related to Node.js releases. and removed lib / src Issues and PRs related to general changes in the lib or src directory. labels Jun 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release Issues and PRs related to Node.js releases.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants