Skip to content

doc: fix typo in embedding.md#64425

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
greenheadHQ:fix-embedding-typo
Jul 18, 2026
Merged

doc: fix typo in embedding.md#64425
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
greenheadHQ:fix-embedding-typo

Conversation

@greenheadHQ

Copy link
Copy Markdown
Contributor

Fixes a typo: Setup upSet up in doc/api/embedding.md (the C++ embedder API example).

-  // Setup up a libuv event loop, v8::Isolate, and Node.js Environment.
+  // Set up a libuv event loop, v8::Isolate, and Node.js Environment.

Set up is the correct verb form (setup is the noun/adjective form), and it also matches the existing wording a few lines above: "In order to set up a v8::Isolate".

Signed-off-by: greenhead <shren0812@gmail.com>
@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. embedding Issues and PRs related to embedding Node.js in another project. labels Jul 11, 2026
@trivikr trivikr added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jul 12, 2026
@greenheadHQ

Copy link
Copy Markdown
Contributor Author

Friendly ping. This is my first contribution here. It has two approvals and the author ready label, and has been open for about a week. Could a collaborator land it when they get a chance? Thanks!

@lpinca lpinca added the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 18, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 18, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 3c88785 into nodejs:main Jul 18, 2026
32 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 3c88785

aduh95 pushed a commit that referenced this pull request Jul 21, 2026
Signed-off-by: greenhead <shren0812@gmail.com>
PR-URL: #64425
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
aduh95 pushed a commit that referenced this pull request Jul 21, 2026
Signed-off-by: greenhead <shren0812@gmail.com>
PR-URL: #64425
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Daeyeon Jeong <daeyeon.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. doc Issues and PRs related to the documentations. embedding Issues and PRs related to embedding Node.js in another project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants