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

[backport 6.x] doc: fix types and description for dns.resolveTxt by tniessen · Pull Request #16316 · nodejs/node · GitHub

/ node Public

[backport 6.x] doc: fix types and description for dns.resolveTxt - #16316

Closed
tniessen wants to merge 1 commit into
nodejs:v6.x-stagingfrom
tniessen:backport-15472-to-v6.x
Closed

[backport 6.x] doc: fix types and description for dns.resolveTxt#16316
tniessen wants to merge 1 commit into
nodejs:v6.x-stagingfrom
tniessen:backport-15472-to-v6.x

Conversation

Copy link
Copy Markdown
Member

PR-URL: #15472

tniessen added the v6.x label Oct 19, 2017
tniessen requested a review from silverwind October 19, 2017 10:38
nodejs-github-bot added dns Issues and PRs related to the dns subsystem. doc Issues and PRs related to the documentations. v6.x labels Oct 19, 2017
Comment thread doc/api/dns.md Outdated

silverwind Oct 19, 2017
edited
Loading

Copy link
Copy Markdown
Contributor

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

I don't think resolveAny was backported to v6.x, so this line probably has to go.

PR-URL: nodejs#15472
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
tniessen force-pushed the backport-15472-to-v6.x branch from f245ee3 to 2192ed7 Compare October 20, 2017 09:13

Copy link
Copy Markdown
Member Author

@silverwind Thank you, fixed! PTAL

MylesBorins pushed a commit that referenced this pull request Oct 25, 2017
Backport-PR-URL: #16316
PR-URL: #15472
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

Copy link
Copy Markdown
Contributor

landed in 3293a0c

MylesBorins pushed a commit that referenced this pull request Oct 25, 2017
Backport-PR-URL: #16316
PR-URL: #15472
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
MylesBorins pushed a commit that referenced this pull request Oct 26, 2017
Backport-PR-URL: #16316
PR-URL: #15472
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
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

dns Issues and PRs related to the dns subsystem. doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL