Skip to content

doc: add documentation for inherited methods - #42691

Merged
nodejs-github-bot merged 1 commit into
nodejs:masterfrom
lpinca:add/inherited-methods
Apr 18, 2022
Merged

doc: add documentation for inherited methods#42691
nodejs-github-bot merged 1 commit into
nodejs:masterfrom
lpinca:add/inherited-methods

Conversation

@lpinca

Copy link
Copy Markdown
Member

These methods are inherited from http.OutgoingMessage and
they are already documented as methods of the http.ServerResponse
class. For consistency, document them also as methods of the
http.ClientRequest class.

These methods are inherited from `http.OutgoingMessage` and
they are already documented as methods of the `http.ServerResponse`
class. For consistency, document them also as methods of the
`http.ClientRequest` class.
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/http
  • @nodejs/net

@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. http Issues or PRs related to the http subsystem. labels Apr 11, 2022

@ShogunPandaShogunPanda left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@crypt096crypt096 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@lpincalpinca added the commit-queue Add this label to land a pull request using GitHub Actions. label Apr 16, 2022
@nodejs-github-botnodejs-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 Apr 16, 2022
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/42691
✔ Done loading data for nodejs/node/pull/42691
----------------------------------- PR info ------------------------------------
Title doc: add documentation for inherited methods (#42691)
⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch lpinca:add/inherited-methods -> nodejs:master
Labels http, doc
Commits 1
- doc: add documentation for inherited methods
Committers 1
- Luigi Pinca PR-URL: https://github.com/nodejs/node/pull/42691
Reviewed-By: Paolo Insogna ------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/42691
Reviewed-By: Paolo Insogna --------------------------------------------------------------------------------
ℹ This PR was created on Mon, 11 Apr 2022 08:49:47 GMT
✔ Approvals: 1
✔ - Paolo Insogna (@ShogunPanda): https://github.com/nodejs/node/pull/42691#pullrequestreview-937686744
✖ This PR needs to wait 37 more hours to land (or 0 hours if there is one more approval)
✔ Last GitHub CI successful
ℹ Green GitHub CI is sufficient
--------------------------------------------------------------------------------
✔ Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/2177682550

@lpincalpinca added the commit-queue Add this label to land a pull request using GitHub Actions. label Apr 18, 2022
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Apr 18, 2022
@nodejs-github-bot
nodejs-github-bot merged commit bde889b into nodejs:masterApr 18, 2022
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in bde889b

@lpinca
lpinca deleted the add/inherited-methods branch April 18, 2022 18:53
xtx1130 pushed a commit to xtx1130/node that referenced this pull request Apr 25, 2022
These methods are inherited from `http.OutgoingMessage` and
they are already documented as methods of the `http.ServerResponse`
class. For consistency, document them also as methods of the
`http.ClientRequest` class.
PR-URL: nodejs#42691
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: Mestery <mestery@protonmail.com>
targos pushed a commit that referenced this pull request Apr 28, 2022
These methods are inherited from `http.OutgoingMessage` and
they are already documented as methods of the `http.ServerResponse`
class. For consistency, document them also as methods of the
`http.ClientRequest` class.
PR-URL: #42691
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: Mestery <mestery@protonmail.com>
@targostargos mentioned this pull request May 2, 2022
juanarbol pushed a commit that referenced this pull request May 31, 2022
These methods are inherited from `http.OutgoingMessage` and
they are already documented as methods of the `http.ServerResponse`
class. For consistency, document them also as methods of the
`http.ClientRequest` class.
PR-URL: #42691
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: Mestery <mestery@protonmail.com>
danielleadams pushed a commit that referenced this pull request Jun 27, 2022
These methods are inherited from `http.OutgoingMessage` and
they are already documented as methods of the `http.ServerResponse`
class. For consistency, document them also as methods of the
`http.ClientRequest` class.
PR-URL: #42691
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: Mestery <mestery@protonmail.com>
targos pushed a commit that referenced this pull request Jul 11, 2022
These methods are inherited from `http.OutgoingMessage` and
they are already documented as methods of the `http.ServerResponse`
class. For consistency, document them also as methods of the
`http.ClientRequest` class.
PR-URL: #42691
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: Mestery <mestery@protonmail.com>
targos pushed a commit that referenced this pull request Jul 31, 2022
These methods are inherited from `http.OutgoingMessage` and
they are already documented as methods of the `http.ServerResponse`
class. For consistency, document them also as methods of the
`http.ClientRequest` class.
PR-URL: #42691
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: Mestery <mestery@protonmail.com>
@targostargos mentioned this pull request Aug 3, 2022
guangwong pushed a commit to noslate-project/node that referenced this pull request Oct 10, 2022
These methods are inherited from `http.OutgoingMessage` and
they are already documented as methods of the `http.ServerResponse`
class. For consistency, document them also as methods of the
`http.ClientRequest` class.
PR-URL: nodejs/node#42691
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: Mestery <mestery@protonmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

commit-queue-failedAn error occurred while landing this pull request using GitHub Actions.docIssues and PRs related to the documentations.httpIssues or PRs related to the http subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@lpinca@nodejs-github-bot@ShogunPanda@crypt096@Mesteery