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

(v6.x-staging) doc: remove leftover WHATWG url.format section by silverwind · Pull Request #14351 · nodejs/node · GitHub

/ node Public

(v6.x-staging) doc: remove leftover WHATWG url.format section - #14351

Closed
silverwind wants to merge 1 commit into
nodejs:v6.x-stagingfrom
silverwind:pr-13054-fix
Closed

(v6.x-staging) doc: remove leftover WHATWG url.format section#14351
silverwind wants to merge 1 commit into
nodejs:v6.x-stagingfrom
silverwind:pr-13054-fix

Conversation

Copy link
Copy Markdown
Contributor

The backport in #13054 contained a leftover section from the WHATWG url.format, in addition to the existing url.format. This removes this section.

Ref: #13054 (comment). Seems to only affect v6.x-staging, v6.x does not have this issue.

cc: @gibfahn

The backport in nodejs#13054 contained a
leftover section from the WHATWG url.format, in addition to the existing
url.format. This removes this section.
nodejs-github-bot added doc Issues and PRs related to the documentations. url Issues and PRs related to the legacy built-in url module. v6.x labels Jul 18, 2017

MylesBorins commented Jul 18, 2017
edited
Loading

Copy link
Copy Markdown
Contributor

landed in fcb27fa

MylesBorins pushed a commit that referenced this pull request Jul 18, 2017
The backport in #13054 contained a
leftover section from the WHATWG url.format, in addition to the existing
url.format. This removes this section.

PR-URL: #14351
Reviewed-By: Myles Borins <myles.borins@gmail.com>
silverwind deleted the pr-13054-fix branch July 18, 2017 18:58
MylesBorins mentioned this pull request Jul 18, 2017
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to the documentations. url Issues and PRs related to the legacy built-in url module.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL