Skip to content

Conversation

@pluris
Copy link
Contributor

A missing parameter was added to the jsdoc of HTTPRequestOptions.

  • joinDuplicateHeaders
  • uniqueHeaders

https://github.com/nodejs/node/blob/main/doc/api/http.md#httprequesturl-options-callback

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/crypto
  • @nodejs/http
  • @nodejs/net

@nodejs-github-botnodejs-github-bot added http Issues or PRs related to the http subsystem. https Issues or PRs related to the https subsystem. needs-ci PRs that need a full CI run. labels Sep 26, 2023
Copy link
Member

@mcollinamcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@mcollinamcollina added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. labels Sep 26, 2023
@github-actionsgithub-actionsbot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 26, 2023
@nodejs-github-bot
Copy link
Collaborator

@aduh95aduh95 added typings and removed needs-ci PRs that need a full CI run. labels Sep 29, 2023
@aduh95aduh95 merged commit 6aa7101 into nodejs:mainSep 29, 2023
@aduh95
Copy link
Contributor

Landed in 6aa7101

alexfernandez pushed a commit to alexfernandez/node that referenced this pull request Nov 1, 2023
PR-URL: nodejs#49872 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Matteo Collina <[email protected]>
@plurispluris deleted the lib/update_jsdoc_http branch November 8, 2023 02:24
targos pushed a commit that referenced this pull request Nov 11, 2023
PR-URL: #49872 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Matteo Collina <[email protected]>
@targostargos mentioned this pull request Nov 12, 2023
@leonardoadame
Copy link

pluris:lib/update_jsdoc_http
#49872 (review)
#49872 (comment)

debadree25 pushed a commit to debadree25/node that referenced this pull request Apr 15, 2024
PR-URL: nodejs#49872 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Matteo Collina <[email protected]>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.httpIssues or PRs related to the http subsystem.httpsIssues or PRs related to the https subsystem.typings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@pluris@nodejs-github-bot@aduh95@leonardoadame@mcollina@marco-ippolito