FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

doc: provide alternative to `url.parse()` using WHATWG URL by styfle · Pull Request #59736 · nodejs/node · GitHub

/ node Public

doc: provide alternative to url.parse() using WHATWG URL - #59736

Closed
styfle wants to merge 2 commits into
nodejs:mainfrom
styfle:styfle/example-url-parse
ConversationCommits (2)ChecksFiles changed
Closed

doc: provide alternative to url.parse() using WHATWG URL#59736
styfle wants to merge 2 commits into
nodejs:mainfrom
styfle:styfle/example-url-parse

Conversation

styfle commented Sep 2, 2025 •
edited
Loading

Copy link
Copy Markdown
Member

nodejs-github-bot added doc Issues and PRs related to the documentations. url Issues and PRs related to the legacy built-in url module. labels Sep 2, 2025
icyJoseph mentioned this pull request Sep 3, 2025
Closed
styfle mentioned this pull request Sep 3, 2025
Merged
Trott reviewed Sep 3, 2025
View reviewed changes
Comment thread doc/api/url.md
Trott approved these changes Sep 3, 2025
View reviewed changes

Trott left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM with or without my nit.

This comment was marked as resolved.

Sign in to view
Flarna reviewed Sep 3, 2025
View reviewed changes
Comment thread doc/api/url.md
styfle requested a review from Flarna September 4, 2025 13:47
sapphi-red reviewed Sep 25, 2025
View reviewed changes
Comment thread doc/api/url.md
anonrig added commit-queue Add this label to land a pull request using GitHub Actions. commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. labels Sep 26, 2025
nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 26, 2025

nodejs-github-bot commented Sep 26, 2025

Copy link
Copy Markdown
Collaborator

Landed in ce72fcc...2e5c8df

nodejs-github-bot pushed a commit that referenced this pull request Sep 26, 2025
doc: provide alternative to url.parse() using WHATWG URL
157cfde 
PR-URL: #59736
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
nodejs-github-bot pushed a commit that referenced this pull request Sep 26, 2025
doc: mention reverse proxy and include simple example
2e5c8df 
PR-URL: #59736
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
targos pushed a commit that referenced this pull request Oct 6, 2025
doc: provide alternative to url.parse() using WHATWG URL
14aa111 
PR-URL: #59736
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
targos pushed a commit that referenced this pull request Oct 6, 2025
doc: mention reverse proxy and include simple example
66d90b8 
PR-URL: #59736
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
github-actions Bot mentioned this pull request Oct 7, 2025
Merged
styfle mentioned this pull request Oct 11, 2025
Merged
styfle deleted the styfle/example-url-parse branch October 11, 2025 01:36
aduh95 pushed a commit that referenced this pull request Oct 11, 2025
doc: mention reverse proxy and include simple example
0a73050 
PR-URL: #59736
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
aduh95 pushed a commit that referenced this pull request Oct 11, 2025
doc: provide alternative to url.parse() using WHATWG URL
13241ba 
PR-URL: #59736
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
aduh95 pushed a commit that referenced this pull request Oct 11, 2025
doc: mention reverse proxy and include simple example
1ddaab1 
PR-URL: #59736
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
aduh95 pushed a commit that referenced this pull request Oct 11, 2025
doc: provide alternative to url.parse() using WHATWG URL
b3eeb3b 
PR-URL: #59736
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
github-actions Bot mentioned this pull request Oct 12, 2025
Merged
slorber mentioned this pull request Nov 20, 2025
Merged
3 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Trott Trott approved these changes
anonrig anonrig approved these changes
trivikr trivikr approved these changes
Flarna Flarna approved these changes
+1 more reviewer
sapphi-red sapphi-red left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. doc Issues and PRs related to the documentations. url Issues and PRs related to the legacy built-in url module.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

Back | FazBrowse Home | New Git URL