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

doc: fix links in Addons docs by estliberitas · Pull Request #5072 · nodejs/node · GitHub

/ node Public

doc: fix links in Addons docs - #5072

Closed
estliberitas wants to merge 1 commit into
nodejs:masterfrom
estliberitas:doc-fix-links-addons
Closed

doc: fix links in Addons docs#5072
estliberitas wants to merge 1 commit into
nodejs:masterfrom
estliberitas:doc-fix-links-addons

Conversation

Copy link
Copy Markdown
Contributor

Put links in a lexical order. Add missing links. Remove duplicates.

Continuing to fix links, their style, etc.

Put links in a lexical order. Add missing links. Remove duplicates.
thefourtheye added the doc Issues and PRs related to the documentations. label Feb 4, 2016

Copy link
Copy Markdown
Contributor

LGTM cc @nodejs/documentation

Comment thread doc/api/addons.markdown
## Addon examples

Following are some example Addons intended to help developers get started. The
examples make use of the V8 APIs. Refer to the online [V8 reference][] for help

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

This change is not necessary.

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

Sorry, read it wrong :)

stevemao commented Feb 4, 2016

Copy link
Copy Markdown
Contributor

Ignore my comments and LGTM.

Copy link
Copy Markdown

LGTM

1 similar comment

jasnell commented Feb 4, 2016

Copy link
Copy Markdown
Member

LGTM

jasnell pushed a commit that referenced this pull request Feb 4, 2016
Put links in a lexical order. Add missing links. Remove duplicates.

PR-URL: #5072
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

jasnell commented Feb 4, 2016

Copy link
Copy Markdown
Member

Landed in 13a3685

jasnell closed this Feb 4, 2016
rvagg pushed a commit that referenced this pull request Feb 8, 2016
Put links in a lexical order. Add missing links. Remove duplicates.

PR-URL: #5072
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
estliberitas deleted the doc-fix-links-addons branch February 14, 2016 17:05
MylesBorins pushed a commit that referenced this pull request Mar 10, 2016
Put links in a lexical order. Add missing links. Remove duplicates.

PR-URL: #5072
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
MylesBorins pushed a commit that referenced this pull request Mar 17, 2016
Put links in a lexical order. Add missing links. Remove duplicates.

PR-URL: #5072
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
MylesBorins pushed a commit that referenced this pull request Mar 21, 2016
Put links in a lexical order. Add missing links. Remove duplicates.

PR-URL: #5072
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
scovetta pushed a commit to scovetta/node that referenced this pull request Apr 2, 2016
Put links in a lexical order. Add missing links. Remove duplicates.

PR-URL: nodejs#5072
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: James M Snell <jasnell@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

doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants


Back | FazBrowse Home | New Git URL