-
Notifications
You must be signed in to change notification settings - Fork 30.7k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
doc: general improvements to punycode.md copy #7025
Conversation
const punycode = require('punycode'); | ||
``` | ||
|
||
[Punycode][] is a character encoding scheme defined by RFC 3492 that is |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Link to RFC 3492
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh. Hmm. I find that a bit confusing, oh well.
Generally LGTM |
ba98639
to
a348b63
Compare
PR-URL: #7025 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Landed in 83e3226 |
PR-URL: #7025 Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Checklist
Affected core subsystem(s)
doc (punycode)
Description of change
General improvements to punycode.md copy
@nodejs/documentation