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

docs: improved & fixed code sample #38776

Closed
wants to merge 4 commits into from

Conversation

todortotev
Copy link
Contributor

Closes #38760

@github-actions github-actions bot added doc Issues and PRs related to the documentations. http Issues or PRs related to the http subsystem. labels May 22, 2021
doc/api/http.md Outdated Show resolved Hide resolved
@todortotev todortotev requested a review from mscdex May 24, 2021 09:34
@aduh95 aduh95 added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label May 24, 2021
@jasnell
Copy link
Member

jasnell commented May 25, 2021

Landed in 5a67f15

@jasnell jasnell closed this May 25, 2021
jasnell pushed a commit that referenced this pull request May 25, 2021
PR-URL: #38776
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
danielleadams pushed a commit that referenced this pull request May 31, 2021
PR-URL: #38776
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
@danielleadams danielleadams mentioned this pull request May 31, 2021
richardlau pushed a commit that referenced this pull request Jul 16, 2021
PR-URL: #38776
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
richardlau pushed a commit that referenced this pull request Jul 19, 2021
PR-URL: #38776
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
richardlau pushed a commit that referenced this pull request Jul 20, 2021
PR-URL: #38776
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
@richardlau richardlau mentioned this pull request Jul 20, 2021
foxxyz pushed a commit to foxxyz/node that referenced this pull request Oct 18, 2021
PR-URL: nodejs#38776
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
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. http Issues or PRs related to the http subsystem.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

doc: (HTTP) http.request(url[, options][, callback]) sample code error
6 participants