Skip to content

Conversation

@Trott
Copy link
Member

Most of the instances of node-addon-api are surrounded with backticks,
but two aren't. Add backticks for consistency.

Checklist

@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. node-api Issues and PRs related to the Node-API. labels Aug 24, 2020
@nodejs-github-bot
Copy link
Collaborator

Review requested:

  • @nodejs/n-api

Copy link
Member

@NickNasoNickNaso left a comment

Choose a reason for hiding this comment

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

LGTM

Most of the instances of _node-addon-api_ are surrounded with backticks, but two aren't. Add backticks for consistency. PR-URL: nodejs#34910 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
@TrottTrott merged commit b01d314 into nodejs:masterAug 27, 2020
@Trott
Copy link
MemberAuthor

Landed in b01d314

@TrottTrott deleted the node-addon-api branch August 27, 2020 13:58
richardlau pushed a commit that referenced this pull request Sep 1, 2020
Most of the instances of _node-addon-api_ are surrounded with backticks, but two aren't. Add backticks for consistency. PR-URL: #34910 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
@richardlaurichardlau mentioned this pull request Sep 2, 2020
4 tasks
addaleax pushed a commit that referenced this pull request Sep 22, 2020
Most of the instances of _node-addon-api_ are surrounded with backticks, but two aren't. Add backticks for consistency. PR-URL: #34910 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
addaleax pushed a commit that referenced this pull request Sep 22, 2020
Most of the instances of _node-addon-api_ are surrounded with backticks, but two aren't. Add backticks for consistency. PR-URL: #34910 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
@codebyterecodebytere mentioned this pull request Sep 28, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docIssues and PRs related to the documentations.node-apiIssues and PRs related to the Node-API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@Trott@nodejs-github-bot@NickNaso@addaleax@lpinca