Skip to content
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: add async_hooks, n-api to _toc.md and all.md #13379

Closed
wants to merge 1 commit into from
Closed

doc: add async_hooks, n-api to _toc.md and all.md #13379

wants to merge 1 commit into from

Conversation

vsemozhetbyt
Copy link
Contributor

Checklist
Affected core subsystem(s)

doc

We still have not N-API in https://nodejs.org/dist/latest-v8.x/docs/api/all.html

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Jun 1, 2017
Copy link
Member

@addaleax addaleax left a comment

Choose a reason for hiding this comment

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

Thank you!

Doesn’t need 48 hours.

@vsemozhetbyt
Copy link
Contributor Author

Landed in 38cec08

@vsemozhetbyt vsemozhetbyt deleted the new-docs branch June 1, 2017 22:17
vsemozhetbyt added a commit that referenced this pull request Jun 1, 2017
PR-URL: #13379
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
@Fishrock123
Copy link
Contributor

Landed before I could approve. 😅

jasnell pushed a commit that referenced this pull request Jun 5, 2017
PR-URL: #13379
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
@jasnell jasnell mentioned this pull request Jun 5, 2017
@gibfahn gibfahn mentioned this pull request Jun 15, 2017
3 tasks
# for free to join this conversation on GitHub. Already have an account? # 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