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

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

Closed

Conversation

aduh95
Copy link
Contributor

@aduh95 aduh95 commented Sep 22, 2024

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 aduh95 added the fast-track PRs that do not need to wait for 48 hours to land. label Sep 22, 2024
Copy link
Contributor

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

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. v20.x v20.x Issues that can be reproduced on v20.x or PRs targeting the v20.x-staging branch. labels Sep 22, 2024
@aduh95 aduh95 changed the title doc: add missing EventSource docs to globals [v20.x backport] doc: add missing EventSource docs to globals Sep 22, 2024
@RedYetiDev RedYetiDev added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. and removed fast-track PRs that do not need to wait for 48 hours to land. labels Sep 22, 2024
@RedYetiDev

This comment was marked as off-topic.

@jakecastelli

This comment was marked as off-topic.

@RedYetiDev

This comment was marked as off-topic.

@jakecastelli

This comment was marked as off-topic.

@aduh95

This comment was marked as off-topic.

targos pushed a commit that referenced this pull request Sep 25, 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
Copy link
Member

targos commented Sep 25, 2024

Landed in 431424a

@targos targos closed this Sep 25, 2024
@aduh95 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>
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 pending requests for changes, and a CI started. doc Issues and PRs related to the documentations. v20.x v20.x Issues that can be reproduced on v20.x or PRs targeting the v20.x-staging branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants