Skip to content

doc: use the WHATWG URL API in http code examples - #29917

Closed
watson wants to merge 1 commit into
nodejs:masterfrom
watson:url-parse
Closed

doc: use the WHATWG URL API in http code examples#29917
watson wants to merge 1 commit into
nodejs:masterfrom
watson:url-parse

Conversation

@watson

Copy link
Copy Markdown
Member
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

@watsonwatson added the fast-track PRs that do not need to wait for 48 hours to land. label Oct 10, 2019
@watsonwatson self-assigned this Oct 10, 2019
@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. http Issues or PRs related to the http subsystem. labels Oct 10, 2019
@watsonwatson added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Oct 10, 2019
@watson

Copy link
Copy Markdown
MemberAuthor

Upvote using 👍 here to approve fast-tracking

Comment threaddoc/api/http.md
@Trott

Copy link
Copy Markdown
Member

Lite CI: https://ci.nodejs.org/job/node-test-pull-request-lite-pipeline/3981/

(@nodejs/github-bot @richardlau It seems like the bot doesn't post the lite CIs automatically anymore for some time. It only posts the full CI runs.)

watson added a commit that referenced this pull request Oct 11, 2019
PR-URL: #29917
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
@watson

Copy link
Copy Markdown
MemberAuthor

Landed in 5909532

@watsonwatson closed this Oct 11, 2019
@watson
watson deleted the url-parse branch October 11, 2019 11:53
@richardlau

Copy link
Copy Markdown
Member

(@nodejs/github-bot @richardlau It seems like the bot doesn't post the lite CIs automatically anymore for some time. It only posts the full CI runs.)

@Trott I've reported it over in nodejs/github-bot#246.

targos pushed a commit that referenced this pull request Oct 14, 2019
PR-URL: #29917
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
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.docIssues and PRs related to the documentations.fast-trackPRs that do not need to wait for 48 hours to land.httpIssues or PRs related to the http subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

11 participants

@watson@Trott@richardlau@jasnell@lpinca@TimothyGu@cjihrig@devnexen@BridgeAR@trivikr@nodejs-github-bot