Skip to content

Conversation

@pluris
Copy link
Contributor

@pluris pluris commented Dec 16, 2023

As mentioned in the problem, the link was incorrect and has been fixed.

Fixes: #51174

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/test_runner

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. test_runner Issues and PRs related to the test runner subsystem. labels Dec 16, 2023
@lpinca
Copy link
Member

lpinca commented Dec 16, 2023

I understand why you added "link" in the commit message but it is a bit confusing. How about "fix TestsStrem parent class"?

@pluris pluris changed the title doc: fix incorrect link for TestsStream doc: fix TestsStream parent class Dec 16, 2023
@pluris
Copy link
Contributor Author

pluris commented Dec 16, 2023

@lpinca I think the commit message you suggested is clearer.
Thank you for the comment. I fixed it.

@aduh95 aduh95 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Dec 17, 2023
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 18, 2023
@nodejs-github-bot nodejs-github-bot merged commit b931e61 into nodejs:main Dec 18, 2023
@nodejs-github-bot
Copy link
Collaborator

Landed in b931e61

RafaelGSS pushed a commit that referenced this pull request Jan 2, 2024
PR-URL: #51181
Fixes: #51174
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Chemi Atlow <[email protected]>
@RafaelGSS RafaelGSS mentioned this pull request Jan 2, 2024
richardlau pushed a commit that referenced this pull request Mar 25, 2024
PR-URL: #51181
Fixes: #51174
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Chemi Atlow <[email protected]>
@richardlau richardlau mentioned this pull request Mar 25, 2024
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. test_runner Issues and PRs related to the test runner subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TestStream is a Node stream but docs link to web ReadableStream

6 participants