Skip to content

Conversation

@Trott
Copy link
Member

@Trott Trott commented Sep 26, 2020

Not a comprehensive edit. Just a few things I noticed.

  • Favor present tense where there's a choice
  • Add a comma for clarity
  • Add a missing word ("in")
Checklist

@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/n-api

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. node-api Issues and PRs related to the Node-API. labels Sep 26, 2020
Not a comprehensive edit. Just a few things I noticed.

* Favor present tense where there's a choice
* Added a comma for clarity
* Added a missing word ("in")

PR-URL: nodejs#35361
Reviewed-By: Chengzhong Wu <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
@Trott
Copy link
Member Author

Trott commented Sep 28, 2020

Landed in 062b35d

@Trott Trott merged commit 062b35d into nodejs:master Sep 28, 2020
@Trott Trott deleted the napiedits branch September 28, 2020 16:29
MylesBorins pushed a commit that referenced this pull request Sep 29, 2020
Not a comprehensive edit. Just a few things I noticed.

* Favor present tense where there's a choice
* Added a comma for clarity
* Added a missing word ("in")

PR-URL: #35361
Reviewed-By: Chengzhong Wu <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
@MylesBorins MylesBorins mentioned this pull request Sep 29, 2020
joesepi pushed a commit to joesepi/node that referenced this pull request Jan 8, 2021
Not a comprehensive edit. Just a few things I noticed.

* Favor present tense where there's a choice
* Added a comma for clarity
* Added a missing word ("in")

PR-URL: nodejs#35361
Reviewed-By: Chengzhong Wu <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
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. node-api Issues and PRs related to the Node-API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants