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

stream: remove whatwg streams experimental warning by jasnell · Pull Request #40971 · nodejs/node · GitHub

/ node Public

stream: remove whatwg streams experimental warning - #40971

Merged
nodejs-github-bot merged 1 commit into
nodejs:masterfrom
jasnell:remove-web-streams-experimental-warning
Dec 1, 2021
Merged

stream: remove whatwg streams experimental warning#40971
nodejs-github-bot merged 1 commit into
nodejs:masterfrom
jasnell:remove-web-streams-experimental-warning

Conversation

jasnell commented Nov 25, 2021

Copy link
Copy Markdown
Member

The API is still experimental, but the warning isn't necessary any longer

Refs: #40950

nodejs-github-bot added needs-ci PRs that need a full CI run. stream Issues and PRs related to the stream subsystem. labels Nov 25, 2021
The API is still experimental, but the warning isn't necessary any
longer

Signed-off-by: James M Snell <jasnell@gmail.com>
jasnell force-pushed the remove-web-streams-experimental-warning branch from a775b03 to 88617b7 Compare November 25, 2021 19:24
targos added the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 30, 2021
nodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Nov 30, 2021

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/40971
✔  Done loading data for nodejs/node/pull/40971
----------------------------------- PR info ------------------------------------
Title      stream: remove whatwg streams experimental warning (#40971)
Author     James M Snell  (@jasnell)
Branch     jasnell:remove-web-streams-experimental-warning -> nodejs:master
Labels     stream, needs-ci
Commits    1
 - stream: remove whatwg streams experimental warning
Committers 1
 - James M Snell 
PR-URL: https://github.com/nodejs/node/pull/40971
Refs: https://github.com/nodejs/node/issues/40950
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: Gerhard Stöbich 
Reviewed-By: Robert Nagy 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/40971
Refs: https://github.com/nodejs/node/issues/40950
Reviewed-By: Benjamin Gruenbaum 
Reviewed-By: Gerhard Stöbich 
Reviewed-By: Robert Nagy 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Thu, 25 Nov 2021 18:54:40 GMT
   ✔  Approvals: 3
   ✔  - Benjamin Gruenbaum (@benjamingr): https://github.com/nodejs/node/pull/40971#pullrequestreview-816291609
   ✔  - Gerhard Stöbich (@Flarna): https://github.com/nodejs/node/pull/40971#pullrequestreview-816306633
   ✔  - Robert Nagy (@ronag) (TSC): https://github.com/nodejs/node/pull/40971#pullrequestreview-816684520
   ✔  Last GitHub Actions successful
   ✖  No Jenkins CI runs detected
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/1522493978

Flarna added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 30, 2021
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 30, 2021

This comment has been minimized.

Copy link
Copy Markdown
Collaborator

targos added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Dec 1, 2021
nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 1, 2021
nodejs-github-bot merged commit 94de738 into nodejs:master Dec 1, 2021

Copy link
Copy Markdown
Collaborator

Landed in 94de738

danielleadams pushed a commit that referenced this pull request Dec 14, 2021
The API is still experimental, but the warning isn't necessary any
longer

Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: #40971
Refs: #40950
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
danielleadams pushed a commit that referenced this pull request Jan 31, 2022
The API is still experimental, but the warning isn't necessary any
longer

Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: #40971
Refs: #40950
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
danielleadams pushed a commit that referenced this pull request Feb 1, 2022
The API is still experimental, but the warning isn't necessary any
longer

Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: #40971
Refs: #40950
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
danielleadams mentioned this pull request Feb 1, 2022
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

needs-ci PRs that need a full CI run. stream Issues and PRs related to the stream subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants


Back | FazBrowse Home | New Git URL