GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — [v24.x] Backport doc deprecations by targos · Pull Request #58542 · nodejs/node

Conversation

@targos

@targos targos commented Jun 1, 2025

Copy link
Copy Markdown
Member

Includes:

  • doc: deprecate HTTP/2 priority signaling
  • doc: clarify DEP0194 scope
  • Doc-only deprecations from lib: deprecate _tls_common and _tls_wrap #57643 and lib: deprecate _stream_* modules #58337

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
mcollina and others added 2 commits June 1, 2025 09:10
@mcollina @aduh95 @targos
doc: deprecate HTTP/2 priority signaling
6cc1d52
Signed-off-by: Matteo Collina <[email protected]>
Co-authored-by: Antoine du Hamel <[email protected]>
PR-URL: nodejs#58313
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Tim Perry <[email protected]>
@aduh95 @targos
doc: clarify DEP0194 scope
0dedb57
Since priority signaling covers more than `http2Stream.priority`,
it makes more sense to rename the section.

There were also a few missing mentions in `http2.md`.

PR-URL: nodejs#58504
Refs: nodejs#58313
Reviewed-By: Tim Perry <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@nodejs-github-bot

nodejs-github-bot commented Jun 1, 2025

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/http
  • @nodejs/http2
  • @nodejs/net
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to Node.js documentation. v24.x Issues that can be reproduced on v24.x or PRs targeting the v24.x-staging branch. labels Jun 1, 2025
@targos targos mentioned this pull request Jun 1, 2025
http2: deprecate priority #58313
Merged
mcollina
mcollina approved these changes Jun 1, 2025
View reviewed changes

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

lgtm

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
richardlau
richardlau approved these changes Jun 1, 2025
View reviewed changes
@richardlau richardlau added lts-watch-v20.x lts-watch-v22.x PRs that may need to be released in v22.x labels Jun 1, 2025
marco-ippolito
marco-ippolito approved these changes Jun 2, 2025
View reviewed changes
RafaelGSS
RafaelGSS approved these changes Jun 2, 2025
View reviewed changes
bjohansebas
bjohansebas approved these changes Jun 3, 2025
View reviewed changes
@targos targos added commit-queue PRs queued for automated landing through the Commit Queue. commit-queue-rebase PRs the Commit Queue should land as multiple self-contained commits. and removed commit-queue PRs queued for automated landing through the Commit Queue. commit-queue-rebase PRs the Commit Queue should land as multiple self-contained commits. labels Jun 3, 2025
targos pushed a commit that referenced this pull request Jun 3, 2025
@mcollina @aduh95 @targos
doc: deprecate HTTP/2 priority signaling
af0446e
Signed-off-by: Matteo Collina <[email protected]>
Co-authored-by: Antoine du Hamel <[email protected]>
PR-URL: #58313
Backport-PR-URL: #58542
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Tim Perry <[email protected]>
targos pushed a commit that referenced this pull request Jun 3, 2025
@aduh95 @targos
doc: clarify DEP0194 scope
2f555e0
Since priority signaling covers more than `http2Stream.priority`,
it makes more sense to rename the section.

There were also a few missing mentions in `http2.md`.

PR-URL: #58504
Backport-PR-URL: #58542
Refs: #58313
Reviewed-By: Tim Perry <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@targos

targos commented Jun 3, 2025

Copy link
Copy Markdown
Member Author

Landed in 8a1eaea...2f555e0

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@targos targos closed this Jun 3, 2025
@targos
targos deleted the backport-deprecation-v24 branch June 3, 2025 08:02
aduh95 added a commit that referenced this pull request Jun 10, 2025
@mcollina @aduh95
doc: deprecate HTTP/2 priority signaling
1e6d7da
Signed-off-by: Matteo Collina <[email protected]>
Co-authored-by: Antoine du Hamel <[email protected]>
PR-URL: #58313
Backport-PR-URL: #58542
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Tim Perry <[email protected]>
aduh95 added a commit that referenced this pull request Jun 10, 2025
@aduh95
doc: clarify DEP0194 scope
7d0df64
Since priority signaling covers more than `http2Stream.priority`,
it makes more sense to rename the section.

There were also a few missing mentions in `http2.md`.

PR-URL: #58504
Backport-PR-URL: #58542
Refs: #58313
Reviewed-By: Tim Perry <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@aduh95 aduh95 added backported-to-v22.x PRs backported to the v22.x-staging branch. and removed lts-watch-v22.x PRs that may need to be released in v22.x labels Jun 10, 2025
marco-ippolito pushed a commit that referenced this pull request Nov 19, 2025
@mcollina @aduh95 @marco-ippolito
doc: deprecate HTTP/2 priority signaling
db68cec
Signed-off-by: Matteo Collina <[email protected]>
Co-authored-by: Antoine du Hamel <[email protected]>
PR-URL: #58313
Backport-PR-URL: #58542
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Tim Perry <[email protected]>
marco-ippolito pushed a commit that referenced this pull request Nov 19, 2025
@aduh95 @marco-ippolito
doc: clarify DEP0194 scope
19a6636
Since priority signaling covers more than `http2Stream.priority`,
it makes more sense to rename the section.

There were also a few missing mentions in `http2.md`.

PR-URL: #58504
Backport-PR-URL: #58542
Refs: #58313
Reviewed-By: Tim Perry <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@marco-ippolito marco-ippolito removed the lts-watch-v20.x label Nov 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mcollina mcollina mcollina approved these changes
@richardlau richardlau richardlau approved these changes
@RafaelGSS RafaelGSS RafaelGSS approved these changes
@marco-ippolito marco-ippolito marco-ippolito approved these changes
+1 more reviewer
@bjohansebas bjohansebas bjohansebas approved these changes
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

backported-to-v22.x PRs backported to the v22.x-staging branch. doc Issues and PRs related to Node.js documentation. v24.x Issues that can be reproduced on v24.x or PRs targeting the v24.x-staging branch.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

8 participants

@targos @nodejs-github-bot @mcollina @richardlau @RafaelGSS @marco-ippolito @bjohansebas @aduh95
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.