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

[v20.x backport] doc: add missing EventSource docs to globals by aduh95 · Pull Request #55059 · nodejs/node · GitHub

/ node Public

[v20.x backport] doc: add missing EventSource docs to globals - #55059

Closed
aduh95 wants to merge 1 commit into
nodejs:v20.x-stagingfrom
aduh95:backport-doc-eventsource-global
Closed

[v20.x backport] doc: add missing EventSource docs to globals#55059
aduh95 wants to merge 1 commit into
nodejs:v20.x-stagingfrom
aduh95:backport-doc-eventsource-global

Conversation

aduh95 commented Sep 22, 2024

Copy link
Copy Markdown
Contributor

Backport of #55022 to fix the added in version. I'm requesting a fast-track so it can be added to the next v20.x release (the new global has already been added to the staging branch) since it's a doc-only change and IMO it makes more sense to not respect the maturation period in this case.

PR-URL: nodejs#55022
Refs: nodejs#51575
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jake Yuesong Li <jake.yuesong@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
aduh95 added the fast-track PRs that do not need to wait for 48 hours to land. label Sep 22, 2024

Copy link
Copy Markdown
Contributor

Fast-track has been requested by @aduh95. Please 👍 to approve.

nodejs-github-bot added doc Issues and PRs related to the documentations. v20.x labels Sep 22, 2024
aduh95 changed the title doc: add missing EventSource docs to globals [v20.x backport] doc: add missing EventSource docs to globals Sep 22, 2024
avivkeller added author ready PRs that have at least one approval, no outstanding review comments, and a CI started. and removed fast-track PRs that do not need to wait for 48 hours to land. labels Sep 22, 2024

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

targos commented Sep 25, 2024

Copy link
Copy Markdown
Member

Landed in 431424a129f2

targos closed this Sep 25, 2024
aduh95 deleted the backport-doc-eventsource-global branch September 25, 2024 20:42
targos pushed a commit that referenced this pull request Sep 26, 2024
PR-URL: #55022
Backport-PR-URL: #55059
Refs: #51575
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jake Yuesong Li <jake.yuesong@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
targos pushed a commit that referenced this pull request Oct 2, 2024
PR-URL: #55022
Backport-PR-URL: #55059
Refs: #51575
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jake Yuesong Li <jake.yuesong@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
targos pushed a commit that referenced this pull request Oct 2, 2024
PR-URL: #55022
Backport-PR-URL: #55059
Refs: #51575
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jake Yuesong Li <jake.yuesong@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
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

author ready PRs that have at least one approval, no outstanding review comments, and a CI started. doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants


Back | FazBrowse Home | New Git URL