doc: remove obsolete --napi-modules doc entry - #64220

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:cli-napi-modules
Jul 7, 2026
Merged

doc: remove obsolete --napi-modules doc entry#64220
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:cli-napi-modules

Conversation

@legendecas

Copy link
Copy Markdown
Member

--napi-modules has been a no-op for a long time. Remove it from the doc.

It is still kept as a no-op option.

@nodejs-github-botnodejs-github-bot added the needs-ci PRs that need a full CI run. label Jun 30, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
@legendecaslegendecas added the node-api Issues and PRs related to Node-API. label Jun 30, 2026
@legendecaslegendecas moved this from Need Triage to In Progress in Node-API Team ProjectJun 30, 2026
@geeksilva97geeksilva97 added author ready PRs with CI started, the required approvals, and no outstanding review comments. request-ci Add this label to start a Jenkins CI on a PR. labels Jun 30, 2026
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 1, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@legendecaslegendecas added the commit-queue PRs queued for automated landing through the Commit Queue. label Jul 2, 2026
@nodejs-github-botnodejs-github-bot added commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. and removed commit-queue PRs queued for automated landing through the Commit Queue. labels Jul 2, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/64220
✔ Done loading data for nodejs/node/pull/64220
----------------------------------- PR info ------------------------------------
Title doc: remove obsolete --napi-modules doc entry (#64220)
⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch legendecas:cli-napi-modules -> nodejs:main
Labels node-api, author ready, needs-ci
Commits 1
- doc: remove obsolete --napi-modules doc entry
Committers 1
- Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: https://github.com/nodejs/node/pull/64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
--------------------------------------------------------------------------------
ℹ This PR was created on Tue, 30 Jun 2026 18:52:29 GMT
✔ Approvals: 4
✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/64220#pullrequestreview-4603771320
✔ - Adrian Estrada (@edsadr): https://github.com/nodejs/node/pull/64220#pullrequestreview-4603915973
✔ - Edy Silva (@geeksilva97): https://github.com/nodejs/node/pull/64220#pullrequestreview-4604828357
✔ - Harshitha K P (@HarshithaKP): https://github.com/nodejs/node/pull/64220#pullrequestreview-4610678742
✔ Last GitHub CI successful
ℹ Last Full PR CI on 2026-07-01T14:52:45Z: https://ci.nodejs.org/job/node-test-pull-request/74493/
- Querying data for job/node-test-pull-request/74493/
✔ Build data downloaded
✔ Last Jenkins CI successful
--------------------------------------------------------------------------------
✔ No git cherry-pick in progress
✔ No git am in progress
✔ No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/main up to date...
From https://github.com/nodejs/node
* branch main -> FETCH_HEAD
✔ origin/main is now up-to-date
- Downloading patch for 64220
From https://github.com/nodejs/node
* branch refs/pull/64220/merge -> FETCH_HEAD
✔ Fetched commits as b2e83150cb54..94da5035468e
--------------------------------------------------------------------------------
Auto-merging doc/api/cli.md
[main 41b4e7f163] doc: remove obsolete --napi-modules doc entry
Author: Chengzhong Wu <cwu631@bloomberg.net>
Date: Tue Jun 30 14:49:37 2026 -0400
4 files changed, 2 insertions(+), 15 deletions(-)
✔ Patches applied
--------------------------------------------------------------------------------
--------------------------------- New Message ----------------------------------
doc: remove obsolete --napi-modules doc entry

Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>

[main 1360e63644] doc: remove obsolete --napi-modules doc entry
Author: Chengzhong Wu <cwu631@bloomberg.net>
Date: Tue Jun 30 14:49:37 2026 -0400
4 files changed, 2 insertions(+), 15 deletions(-)
✔ 1360e6364443ed602d0e4dff94ebc33a6c0aae4f
✔ 0:0 no Assisted-by metadata assisted-by-is-trailer
✔ 0:0 no Co-authored-by metadata co-authored-by-is-trailer
✔ 0:0 skipping fixes-url fixes-url
✔ 0:0 blank line after title line-after-title
✔ 0:0 line-lengths are valid line-length
✔ 0:0 metadata is at end of message metadata-end
✔ 2:8 PR-URL is valid. pr-url
✔ 0:0 reviewers are valid reviewers
⚠ 1:0 "Signed-off-by" email does not match the commit author email signed-off-by
✔ 0:0 valid subsystems subsystem
✔ 0:0 Title is formatted correctly. title-format
✔ 0:0 Title is <= 50 columns. title-length

The following commits are landable on origin/main

  • 1360e63644 doc: remove obsolete --napi-modules doc entry

  • Removing temporary files
    ✔ Temporary files removed.
    To finish landing:
  1. Run:
    git push origin main
  2. Post "Landed in 1360e6364443" in doc: remove obsolete --napi-modules doc entry #64220
    gh pr comment doc: remove obsolete --napi-modules doc entry #64220 --body "Landed in 1360e6364443"
    gh pr close doc: remove obsolete --napi-modules doc entry #64220
https://github.com/nodejs/node/actions/runs/28615542785

@legendecaslegendecas added commit-queue PRs queued for automated landing through the Commit Queue. and removed commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. labels Jul 7, 2026
@nodejs-github-botnodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Jul 7, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 94164da into nodejs:mainJul 7, 2026
84 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 94164da

@github-project-automationgithub-project-automationBot moved this from In Progress to Done in Node-API Team ProjectJul 7, 2026
@legendecas
legendecas deleted the cli-napi-modules branch July 7, 2026 16:36
aduh95 pushed a commit that referenced this pull request Jul 21, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Jul 29, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
aduh95 pushed a commit that referenced this pull request Aug 6, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs with CI started, the required approvals, and no outstanding review comments.needs-ciPRs that need a full CI run.node-apiIssues and PRs related to Node-API.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants

@legendecas@nodejs-github-bot@edsadr@lpinca@geeksilva97@HarshithaKP
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content

doc: remove obsolete --napi-modules doc entry - #64220

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:cli-napi-modules
Jul 7, 2026
Merged

doc: remove obsolete --napi-modules doc entry#64220
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:cli-napi-modules

Conversation

@legendecas

Copy link
Copy Markdown
Member

--napi-modules has been a no-op for a long time. Remove it from the doc.

It is still kept as a no-op option.

@nodejs-github-botnodejs-github-bot added the needs-ci PRs that need a full CI run. label Jun 30, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
@legendecaslegendecas added the node-api Issues and PRs related to Node-API. label Jun 30, 2026
@legendecaslegendecas moved this from Need Triage to In Progress in Node-API Team ProjectJun 30, 2026
@geeksilva97geeksilva97 added author ready PRs with CI started, the required approvals, and no outstanding review comments. request-ci Add this label to start a Jenkins CI on a PR. labels Jun 30, 2026
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 1, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@legendecaslegendecas added the commit-queue PRs queued for automated landing through the Commit Queue. label Jul 2, 2026
@nodejs-github-botnodejs-github-bot added commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. and removed commit-queue PRs queued for automated landing through the Commit Queue. labels Jul 2, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/64220
✔ Done loading data for nodejs/node/pull/64220
----------------------------------- PR info ------------------------------------
Title doc: remove obsolete --napi-modules doc entry (#64220)
⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch legendecas:cli-napi-modules -> nodejs:main
Labels node-api, author ready, needs-ci
Commits 1
- doc: remove obsolete --napi-modules doc entry
Committers 1
- Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: https://github.com/nodejs/node/pull/64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
--------------------------------------------------------------------------------
ℹ This PR was created on Tue, 30 Jun 2026 18:52:29 GMT
✔ Approvals: 4
✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/64220#pullrequestreview-4603771320
✔ - Adrian Estrada (@edsadr): https://github.com/nodejs/node/pull/64220#pullrequestreview-4603915973
✔ - Edy Silva (@geeksilva97): https://github.com/nodejs/node/pull/64220#pullrequestreview-4604828357
✔ - Harshitha K P (@HarshithaKP): https://github.com/nodejs/node/pull/64220#pullrequestreview-4610678742
✔ Last GitHub CI successful
ℹ Last Full PR CI on 2026-07-01T14:52:45Z: https://ci.nodejs.org/job/node-test-pull-request/74493/
- Querying data for job/node-test-pull-request/74493/
✔ Build data downloaded
✔ Last Jenkins CI successful
--------------------------------------------------------------------------------
✔ No git cherry-pick in progress
✔ No git am in progress
✔ No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/main up to date...
From https://github.com/nodejs/node
* branch main -> FETCH_HEAD
✔ origin/main is now up-to-date
- Downloading patch for 64220
From https://github.com/nodejs/node
* branch refs/pull/64220/merge -> FETCH_HEAD
✔ Fetched commits as b2e83150cb54..94da5035468e
--------------------------------------------------------------------------------
Auto-merging doc/api/cli.md
[main 41b4e7f163] doc: remove obsolete --napi-modules doc entry
Author: Chengzhong Wu <cwu631@bloomberg.net>
Date: Tue Jun 30 14:49:37 2026 -0400
4 files changed, 2 insertions(+), 15 deletions(-)
✔ Patches applied
--------------------------------------------------------------------------------
--------------------------------- New Message ----------------------------------
doc: remove obsolete --napi-modules doc entry

Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>

[main 1360e63644] doc: remove obsolete --napi-modules doc entry
Author: Chengzhong Wu <cwu631@bloomberg.net>
Date: Tue Jun 30 14:49:37 2026 -0400
4 files changed, 2 insertions(+), 15 deletions(-)
✔ 1360e6364443ed602d0e4dff94ebc33a6c0aae4f
✔ 0:0 no Assisted-by metadata assisted-by-is-trailer
✔ 0:0 no Co-authored-by metadata co-authored-by-is-trailer
✔ 0:0 skipping fixes-url fixes-url
✔ 0:0 blank line after title line-after-title
✔ 0:0 line-lengths are valid line-length
✔ 0:0 metadata is at end of message metadata-end
✔ 2:8 PR-URL is valid. pr-url
✔ 0:0 reviewers are valid reviewers
⚠ 1:0 "Signed-off-by" email does not match the commit author email signed-off-by
✔ 0:0 valid subsystems subsystem
✔ 0:0 Title is formatted correctly. title-format
✔ 0:0 Title is <= 50 columns. title-length

The following commits are landable on origin/main

  • 1360e63644 doc: remove obsolete --napi-modules doc entry

  • Removing temporary files
    ✔ Temporary files removed.
    To finish landing:
  1. Run:
    git push origin main
  2. Post "Landed in 1360e6364443" in doc: remove obsolete --napi-modules doc entry #64220
    gh pr comment doc: remove obsolete --napi-modules doc entry #64220 --body "Landed in 1360e6364443"
    gh pr close doc: remove obsolete --napi-modules doc entry #64220
https://github.com/nodejs/node/actions/runs/28615542785

@legendecaslegendecas added commit-queue PRs queued for automated landing through the Commit Queue. and removed commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. labels Jul 7, 2026
@nodejs-github-botnodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Jul 7, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 94164da into nodejs:mainJul 7, 2026
84 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 94164da

@github-project-automationgithub-project-automationBot moved this from In Progress to Done in Node-API Team ProjectJul 7, 2026
@legendecas
legendecas deleted the cli-napi-modules branch July 7, 2026 16:36
aduh95 pushed a commit that referenced this pull request Jul 21, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Jul 29, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
aduh95 pushed a commit that referenced this pull request Aug 6, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs with CI started, the required approvals, and no outstanding review comments.needs-ciPRs that need a full CI run.node-apiIssues and PRs related to Node-API.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants

@legendecas@nodejs-github-bot@edsadr@lpinca@geeksilva97@HarshithaKP
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

doc: remove obsolete --napi-modules doc entry - #64220

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:cli-napi-modules
Jul 7, 2026
Merged

doc: remove obsolete --napi-modules doc entry#64220
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:cli-napi-modules

Conversation

@legendecas

Copy link
Copy Markdown
Member

--napi-modules has been a no-op for a long time. Remove it from the doc.

It is still kept as a no-op option.

@nodejs-github-botnodejs-github-bot added the needs-ci PRs that need a full CI run. label Jun 30, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
@legendecaslegendecas added the node-api Issues and PRs related to Node-API. label Jun 30, 2026
@legendecaslegendecas moved this from Need Triage to In Progress in Node-API Team ProjectJun 30, 2026
@geeksilva97geeksilva97 added author ready PRs with CI started, the required approvals, and no outstanding review comments. request-ci Add this label to start a Jenkins CI on a PR. labels Jun 30, 2026
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 1, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@legendecaslegendecas added the commit-queue PRs queued for automated landing through the Commit Queue. label Jul 2, 2026
@nodejs-github-botnodejs-github-bot added commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. and removed commit-queue PRs queued for automated landing through the Commit Queue. labels Jul 2, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/64220
✔ Done loading data for nodejs/node/pull/64220
----------------------------------- PR info ------------------------------------
Title doc: remove obsolete --napi-modules doc entry (#64220)
⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch legendecas:cli-napi-modules -> nodejs:main
Labels node-api, author ready, needs-ci
Commits 1
- doc: remove obsolete --napi-modules doc entry
Committers 1
- Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: https://github.com/nodejs/node/pull/64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
--------------------------------------------------------------------------------
ℹ This PR was created on Tue, 30 Jun 2026 18:52:29 GMT
✔ Approvals: 4
✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/64220#pullrequestreview-4603771320
✔ - Adrian Estrada (@edsadr): https://github.com/nodejs/node/pull/64220#pullrequestreview-4603915973
✔ - Edy Silva (@geeksilva97): https://github.com/nodejs/node/pull/64220#pullrequestreview-4604828357
✔ - Harshitha K P (@HarshithaKP): https://github.com/nodejs/node/pull/64220#pullrequestreview-4610678742
✔ Last GitHub CI successful
ℹ Last Full PR CI on 2026-07-01T14:52:45Z: https://ci.nodejs.org/job/node-test-pull-request/74493/
- Querying data for job/node-test-pull-request/74493/
✔ Build data downloaded
✔ Last Jenkins CI successful
--------------------------------------------------------------------------------
✔ No git cherry-pick in progress
✔ No git am in progress
✔ No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/main up to date...
From https://github.com/nodejs/node
* branch main -> FETCH_HEAD
✔ origin/main is now up-to-date
- Downloading patch for 64220
From https://github.com/nodejs/node
* branch refs/pull/64220/merge -> FETCH_HEAD
✔ Fetched commits as b2e83150cb54..94da5035468e
--------------------------------------------------------------------------------
Auto-merging doc/api/cli.md
[main 41b4e7f163] doc: remove obsolete --napi-modules doc entry
Author: Chengzhong Wu <cwu631@bloomberg.net>
Date: Tue Jun 30 14:49:37 2026 -0400
4 files changed, 2 insertions(+), 15 deletions(-)
✔ Patches applied
--------------------------------------------------------------------------------
--------------------------------- New Message ----------------------------------
doc: remove obsolete --napi-modules doc entry

Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>

[main 1360e63644] doc: remove obsolete --napi-modules doc entry
Author: Chengzhong Wu <cwu631@bloomberg.net>
Date: Tue Jun 30 14:49:37 2026 -0400
4 files changed, 2 insertions(+), 15 deletions(-)
✔ 1360e6364443ed602d0e4dff94ebc33a6c0aae4f
✔ 0:0 no Assisted-by metadata assisted-by-is-trailer
✔ 0:0 no Co-authored-by metadata co-authored-by-is-trailer
✔ 0:0 skipping fixes-url fixes-url
✔ 0:0 blank line after title line-after-title
✔ 0:0 line-lengths are valid line-length
✔ 0:0 metadata is at end of message metadata-end
✔ 2:8 PR-URL is valid. pr-url
✔ 0:0 reviewers are valid reviewers
⚠ 1:0 "Signed-off-by" email does not match the commit author email signed-off-by
✔ 0:0 valid subsystems subsystem
✔ 0:0 Title is formatted correctly. title-format
✔ 0:0 Title is <= 50 columns. title-length

The following commits are landable on origin/main

  • 1360e63644 doc: remove obsolete --napi-modules doc entry

  • Removing temporary files
    ✔ Temporary files removed.
    To finish landing:
  1. Run:
    git push origin main
  2. Post "Landed in 1360e6364443" in doc: remove obsolete --napi-modules doc entry #64220
    gh pr comment doc: remove obsolete --napi-modules doc entry #64220 --body "Landed in 1360e6364443"
    gh pr close doc: remove obsolete --napi-modules doc entry #64220
https://github.com/nodejs/node/actions/runs/28615542785

@legendecaslegendecas added commit-queue PRs queued for automated landing through the Commit Queue. and removed commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. labels Jul 7, 2026
@nodejs-github-botnodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Jul 7, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 94164da into nodejs:mainJul 7, 2026
84 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 94164da

@github-project-automationgithub-project-automationBot moved this from In Progress to Done in Node-API Team ProjectJul 7, 2026
@legendecas
legendecas deleted the cli-napi-modules branch July 7, 2026 16:36
aduh95 pushed a commit that referenced this pull request Jul 21, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Jul 29, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
aduh95 pushed a commit that referenced this pull request Aug 6, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs with CI started, the required approvals, and no outstanding review comments.needs-ciPRs that need a full CI run.node-apiIssues and PRs related to Node-API.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants

@legendecas@nodejs-github-bot@edsadr@lpinca@geeksilva97@HarshithaKP
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

doc: remove obsolete --napi-modules doc entry - #64220

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:cli-napi-modules
Jul 7, 2026
Merged

doc: remove obsolete --napi-modules doc entry#64220
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:cli-napi-modules

Conversation

@legendecas

Copy link
Copy Markdown
Member

--napi-modules has been a no-op for a long time. Remove it from the doc.

It is still kept as a no-op option.

@nodejs-github-botnodejs-github-bot added the needs-ci PRs that need a full CI run. label Jun 30, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
@legendecaslegendecas added the node-api Issues and PRs related to Node-API. label Jun 30, 2026
@legendecaslegendecas moved this from Need Triage to In Progress in Node-API Team ProjectJun 30, 2026
@geeksilva97geeksilva97 added author ready PRs with CI started, the required approvals, and no outstanding review comments. request-ci Add this label to start a Jenkins CI on a PR. labels Jun 30, 2026
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 1, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@legendecaslegendecas added the commit-queue PRs queued for automated landing through the Commit Queue. label Jul 2, 2026
@nodejs-github-botnodejs-github-bot added commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. and removed commit-queue PRs queued for automated landing through the Commit Queue. labels Jul 2, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/64220
✔ Done loading data for nodejs/node/pull/64220
----------------------------------- PR info ------------------------------------
Title doc: remove obsolete --napi-modules doc entry (#64220)
⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch legendecas:cli-napi-modules -> nodejs:main
Labels node-api, author ready, needs-ci
Commits 1
- doc: remove obsolete --napi-modules doc entry
Committers 1
- Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: https://github.com/nodejs/node/pull/64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
--------------------------------------------------------------------------------
ℹ This PR was created on Tue, 30 Jun 2026 18:52:29 GMT
✔ Approvals: 4
✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/64220#pullrequestreview-4603771320
✔ - Adrian Estrada (@edsadr): https://github.com/nodejs/node/pull/64220#pullrequestreview-4603915973
✔ - Edy Silva (@geeksilva97): https://github.com/nodejs/node/pull/64220#pullrequestreview-4604828357
✔ - Harshitha K P (@HarshithaKP): https://github.com/nodejs/node/pull/64220#pullrequestreview-4610678742
✔ Last GitHub CI successful
ℹ Last Full PR CI on 2026-07-01T14:52:45Z: https://ci.nodejs.org/job/node-test-pull-request/74493/
- Querying data for job/node-test-pull-request/74493/
✔ Build data downloaded
✔ Last Jenkins CI successful
--------------------------------------------------------------------------------
✔ No git cherry-pick in progress
✔ No git am in progress
✔ No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/main up to date...
From https://github.com/nodejs/node
* branch main -> FETCH_HEAD
✔ origin/main is now up-to-date
- Downloading patch for 64220
From https://github.com/nodejs/node
* branch refs/pull/64220/merge -> FETCH_HEAD
✔ Fetched commits as b2e83150cb54..94da5035468e
--------------------------------------------------------------------------------
Auto-merging doc/api/cli.md
[main 41b4e7f163] doc: remove obsolete --napi-modules doc entry
Author: Chengzhong Wu <cwu631@bloomberg.net>
Date: Tue Jun 30 14:49:37 2026 -0400
4 files changed, 2 insertions(+), 15 deletions(-)
✔ Patches applied
--------------------------------------------------------------------------------
--------------------------------- New Message ----------------------------------
doc: remove obsolete --napi-modules doc entry

Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>

[main 1360e63644] doc: remove obsolete --napi-modules doc entry
Author: Chengzhong Wu <cwu631@bloomberg.net>
Date: Tue Jun 30 14:49:37 2026 -0400
4 files changed, 2 insertions(+), 15 deletions(-)
✔ 1360e6364443ed602d0e4dff94ebc33a6c0aae4f
✔ 0:0 no Assisted-by metadata assisted-by-is-trailer
✔ 0:0 no Co-authored-by metadata co-authored-by-is-trailer
✔ 0:0 skipping fixes-url fixes-url
✔ 0:0 blank line after title line-after-title
✔ 0:0 line-lengths are valid line-length
✔ 0:0 metadata is at end of message metadata-end
✔ 2:8 PR-URL is valid. pr-url
✔ 0:0 reviewers are valid reviewers
⚠ 1:0 "Signed-off-by" email does not match the commit author email signed-off-by
✔ 0:0 valid subsystems subsystem
✔ 0:0 Title is formatted correctly. title-format
✔ 0:0 Title is <= 50 columns. title-length

The following commits are landable on origin/main

  • 1360e63644 doc: remove obsolete --napi-modules doc entry

  • Removing temporary files
    ✔ Temporary files removed.
    To finish landing:
  1. Run:
    git push origin main
  2. Post "Landed in 1360e6364443" in doc: remove obsolete --napi-modules doc entry #64220
    gh pr comment doc: remove obsolete --napi-modules doc entry #64220 --body "Landed in 1360e6364443"
    gh pr close doc: remove obsolete --napi-modules doc entry #64220
https://github.com/nodejs/node/actions/runs/28615542785

@legendecaslegendecas added commit-queue PRs queued for automated landing through the Commit Queue. and removed commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. labels Jul 7, 2026
@nodejs-github-botnodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Jul 7, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 94164da into nodejs:mainJul 7, 2026
84 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 94164da

@github-project-automationgithub-project-automationBot moved this from In Progress to Done in Node-API Team ProjectJul 7, 2026
@legendecas
legendecas deleted the cli-napi-modules branch July 7, 2026 16:36
aduh95 pushed a commit that referenced this pull request Jul 21, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Jul 29, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
aduh95 pushed a commit that referenced this pull request Aug 6, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs with CI started, the required approvals, and no outstanding review comments.needs-ciPRs that need a full CI run.node-apiIssues and PRs related to Node-API.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants

@legendecas@nodejs-github-bot@edsadr@lpinca@geeksilva97@HarshithaKP
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content

doc: remove obsolete --napi-modules doc entry - #64220

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:cli-napi-modules
Jul 7, 2026
Merged

doc: remove obsolete --napi-modules doc entry#64220
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:cli-napi-modules

Conversation

@legendecas

Copy link
Copy Markdown
Member

--napi-modules has been a no-op for a long time. Remove it from the doc.

It is still kept as a no-op option.

@nodejs-github-botnodejs-github-bot added the needs-ci PRs that need a full CI run. label Jun 30, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
@legendecaslegendecas added the node-api Issues and PRs related to Node-API. label Jun 30, 2026
@legendecaslegendecas moved this from Need Triage to In Progress in Node-API Team ProjectJun 30, 2026
@geeksilva97geeksilva97 added author ready PRs with CI started, the required approvals, and no outstanding review comments. request-ci Add this label to start a Jenkins CI on a PR. labels Jun 30, 2026
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 1, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@legendecaslegendecas added the commit-queue PRs queued for automated landing through the Commit Queue. label Jul 2, 2026
@nodejs-github-botnodejs-github-bot added commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. and removed commit-queue PRs queued for automated landing through the Commit Queue. labels Jul 2, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/64220
✔ Done loading data for nodejs/node/pull/64220
----------------------------------- PR info ------------------------------------
Title doc: remove obsolete --napi-modules doc entry (#64220)
⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch legendecas:cli-napi-modules -> nodejs:main
Labels node-api, author ready, needs-ci
Commits 1
- doc: remove obsolete --napi-modules doc entry
Committers 1
- Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: https://github.com/nodejs/node/pull/64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
--------------------------------------------------------------------------------
ℹ This PR was created on Tue, 30 Jun 2026 18:52:29 GMT
✔ Approvals: 4
✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/64220#pullrequestreview-4603771320
✔ - Adrian Estrada (@edsadr): https://github.com/nodejs/node/pull/64220#pullrequestreview-4603915973
✔ - Edy Silva (@geeksilva97): https://github.com/nodejs/node/pull/64220#pullrequestreview-4604828357
✔ - Harshitha K P (@HarshithaKP): https://github.com/nodejs/node/pull/64220#pullrequestreview-4610678742
✔ Last GitHub CI successful
ℹ Last Full PR CI on 2026-07-01T14:52:45Z: https://ci.nodejs.org/job/node-test-pull-request/74493/
- Querying data for job/node-test-pull-request/74493/
✔ Build data downloaded
✔ Last Jenkins CI successful
--------------------------------------------------------------------------------
✔ No git cherry-pick in progress
✔ No git am in progress
✔ No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/main up to date...
From https://github.com/nodejs/node
* branch main -> FETCH_HEAD
✔ origin/main is now up-to-date
- Downloading patch for 64220
From https://github.com/nodejs/node
* branch refs/pull/64220/merge -> FETCH_HEAD
✔ Fetched commits as b2e83150cb54..94da5035468e
--------------------------------------------------------------------------------
Auto-merging doc/api/cli.md
[main 41b4e7f163] doc: remove obsolete --napi-modules doc entry
Author: Chengzhong Wu <cwu631@bloomberg.net>
Date: Tue Jun 30 14:49:37 2026 -0400
4 files changed, 2 insertions(+), 15 deletions(-)
✔ Patches applied
--------------------------------------------------------------------------------
--------------------------------- New Message ----------------------------------
doc: remove obsolete --napi-modules doc entry

Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>

[main 1360e63644] doc: remove obsolete --napi-modules doc entry
Author: Chengzhong Wu <cwu631@bloomberg.net>
Date: Tue Jun 30 14:49:37 2026 -0400
4 files changed, 2 insertions(+), 15 deletions(-)
✔ 1360e6364443ed602d0e4dff94ebc33a6c0aae4f
✔ 0:0 no Assisted-by metadata assisted-by-is-trailer
✔ 0:0 no Co-authored-by metadata co-authored-by-is-trailer
✔ 0:0 skipping fixes-url fixes-url
✔ 0:0 blank line after title line-after-title
✔ 0:0 line-lengths are valid line-length
✔ 0:0 metadata is at end of message metadata-end
✔ 2:8 PR-URL is valid. pr-url
✔ 0:0 reviewers are valid reviewers
⚠ 1:0 "Signed-off-by" email does not match the commit author email signed-off-by
✔ 0:0 valid subsystems subsystem
✔ 0:0 Title is formatted correctly. title-format
✔ 0:0 Title is <= 50 columns. title-length

The following commits are landable on origin/main

  • 1360e63644 doc: remove obsolete --napi-modules doc entry

  • Removing temporary files
    ✔ Temporary files removed.
    To finish landing:
  1. Run:
    git push origin main
  2. Post "Landed in 1360e6364443" in doc: remove obsolete --napi-modules doc entry #64220
    gh pr comment doc: remove obsolete --napi-modules doc entry #64220 --body "Landed in 1360e6364443"
    gh pr close doc: remove obsolete --napi-modules doc entry #64220
https://github.com/nodejs/node/actions/runs/28615542785

@legendecaslegendecas added commit-queue PRs queued for automated landing through the Commit Queue. and removed commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. labels Jul 7, 2026
@nodejs-github-botnodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Jul 7, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 94164da into nodejs:mainJul 7, 2026
84 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 94164da

@github-project-automationgithub-project-automationBot moved this from In Progress to Done in Node-API Team ProjectJul 7, 2026
@legendecas
legendecas deleted the cli-napi-modules branch July 7, 2026 16:36
aduh95 pushed a commit that referenced this pull request Jul 21, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Jul 29, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
aduh95 pushed a commit that referenced this pull request Aug 6, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs with CI started, the required approvals, and no outstanding review comments.needs-ciPRs that need a full CI run.node-apiIssues and PRs related to Node-API.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants

@legendecas@nodejs-github-bot@edsadr@lpinca@geeksilva97@HarshithaKP
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

doc: remove obsolete --napi-modules doc entry - #64220

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:cli-napi-modules
Jul 7, 2026
Merged

doc: remove obsolete --napi-modules doc entry#64220
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:cli-napi-modules

Conversation

@legendecas

Copy link
Copy Markdown
Member

--napi-modules has been a no-op for a long time. Remove it from the doc.

It is still kept as a no-op option.

@nodejs-github-botnodejs-github-bot added the needs-ci PRs that need a full CI run. label Jun 30, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
@legendecaslegendecas added the node-api Issues and PRs related to Node-API. label Jun 30, 2026
@legendecaslegendecas moved this from Need Triage to In Progress in Node-API Team ProjectJun 30, 2026
@geeksilva97geeksilva97 added author ready PRs with CI started, the required approvals, and no outstanding review comments. request-ci Add this label to start a Jenkins CI on a PR. labels Jun 30, 2026
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 1, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@legendecaslegendecas added the commit-queue PRs queued for automated landing through the Commit Queue. label Jul 2, 2026
@nodejs-github-botnodejs-github-bot added commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. and removed commit-queue PRs queued for automated landing through the Commit Queue. labels Jul 2, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/64220
✔ Done loading data for nodejs/node/pull/64220
----------------------------------- PR info ------------------------------------
Title doc: remove obsolete --napi-modules doc entry (#64220)
⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch legendecas:cli-napi-modules -> nodejs:main
Labels node-api, author ready, needs-ci
Commits 1
- doc: remove obsolete --napi-modules doc entry
Committers 1
- Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: https://github.com/nodejs/node/pull/64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
--------------------------------------------------------------------------------
ℹ This PR was created on Tue, 30 Jun 2026 18:52:29 GMT
✔ Approvals: 4
✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/64220#pullrequestreview-4603771320
✔ - Adrian Estrada (@edsadr): https://github.com/nodejs/node/pull/64220#pullrequestreview-4603915973
✔ - Edy Silva (@geeksilva97): https://github.com/nodejs/node/pull/64220#pullrequestreview-4604828357
✔ - Harshitha K P (@HarshithaKP): https://github.com/nodejs/node/pull/64220#pullrequestreview-4610678742
✔ Last GitHub CI successful
ℹ Last Full PR CI on 2026-07-01T14:52:45Z: https://ci.nodejs.org/job/node-test-pull-request/74493/
- Querying data for job/node-test-pull-request/74493/
✔ Build data downloaded
✔ Last Jenkins CI successful
--------------------------------------------------------------------------------
✔ No git cherry-pick in progress
✔ No git am in progress
✔ No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/main up to date...
From https://github.com/nodejs/node
* branch main -> FETCH_HEAD
✔ origin/main is now up-to-date
- Downloading patch for 64220
From https://github.com/nodejs/node
* branch refs/pull/64220/merge -> FETCH_HEAD
✔ Fetched commits as b2e83150cb54..94da5035468e
--------------------------------------------------------------------------------
Auto-merging doc/api/cli.md
[main 41b4e7f163] doc: remove obsolete --napi-modules doc entry
Author: Chengzhong Wu <cwu631@bloomberg.net>
Date: Tue Jun 30 14:49:37 2026 -0400
4 files changed, 2 insertions(+), 15 deletions(-)
✔ Patches applied
--------------------------------------------------------------------------------
--------------------------------- New Message ----------------------------------
doc: remove obsolete --napi-modules doc entry

Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>

[main 1360e63644] doc: remove obsolete --napi-modules doc entry
Author: Chengzhong Wu <cwu631@bloomberg.net>
Date: Tue Jun 30 14:49:37 2026 -0400
4 files changed, 2 insertions(+), 15 deletions(-)
✔ 1360e6364443ed602d0e4dff94ebc33a6c0aae4f
✔ 0:0 no Assisted-by metadata assisted-by-is-trailer
✔ 0:0 no Co-authored-by metadata co-authored-by-is-trailer
✔ 0:0 skipping fixes-url fixes-url
✔ 0:0 blank line after title line-after-title
✔ 0:0 line-lengths are valid line-length
✔ 0:0 metadata is at end of message metadata-end
✔ 2:8 PR-URL is valid. pr-url
✔ 0:0 reviewers are valid reviewers
⚠ 1:0 "Signed-off-by" email does not match the commit author email signed-off-by
✔ 0:0 valid subsystems subsystem
✔ 0:0 Title is formatted correctly. title-format
✔ 0:0 Title is <= 50 columns. title-length

The following commits are landable on origin/main

  • 1360e63644 doc: remove obsolete --napi-modules doc entry

  • Removing temporary files
    ✔ Temporary files removed.
    To finish landing:
  1. Run:
    git push origin main
  2. Post "Landed in 1360e6364443" in doc: remove obsolete --napi-modules doc entry #64220
    gh pr comment doc: remove obsolete --napi-modules doc entry #64220 --body "Landed in 1360e6364443"
    gh pr close doc: remove obsolete --napi-modules doc entry #64220
https://github.com/nodejs/node/actions/runs/28615542785

@legendecaslegendecas added commit-queue PRs queued for automated landing through the Commit Queue. and removed commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. labels Jul 7, 2026
@nodejs-github-botnodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Jul 7, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 94164da into nodejs:mainJul 7, 2026
84 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 94164da

@github-project-automationgithub-project-automationBot moved this from In Progress to Done in Node-API Team ProjectJul 7, 2026
@legendecas
legendecas deleted the cli-napi-modules branch July 7, 2026 16:36
aduh95 pushed a commit that referenced this pull request Jul 21, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Jul 29, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
aduh95 pushed a commit that referenced this pull request Aug 6, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs with CI started, the required approvals, and no outstanding review comments.needs-ciPRs that need a full CI run.node-apiIssues and PRs related to Node-API.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants

@legendecas@nodejs-github-bot@edsadr@lpinca@geeksilva97@HarshithaKP
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

doc: remove obsolete --napi-modules doc entry - #64220

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:cli-napi-modules
Jul 7, 2026
Merged

doc: remove obsolete --napi-modules doc entry#64220
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:cli-napi-modules

Conversation

@legendecas

Copy link
Copy Markdown
Member

--napi-modules has been a no-op for a long time. Remove it from the doc.

It is still kept as a no-op option.

@nodejs-github-botnodejs-github-bot added the needs-ci PRs that need a full CI run. label Jun 30, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
@legendecaslegendecas added the node-api Issues and PRs related to Node-API. label Jun 30, 2026
@legendecaslegendecas moved this from Need Triage to In Progress in Node-API Team ProjectJun 30, 2026
@geeksilva97geeksilva97 added author ready PRs with CI started, the required approvals, and no outstanding review comments. request-ci Add this label to start a Jenkins CI on a PR. labels Jun 30, 2026
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 1, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@legendecaslegendecas added the commit-queue PRs queued for automated landing through the Commit Queue. label Jul 2, 2026
@nodejs-github-botnodejs-github-bot added commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. and removed commit-queue PRs queued for automated landing through the Commit Queue. labels Jul 2, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/64220
✔ Done loading data for nodejs/node/pull/64220
----------------------------------- PR info ------------------------------------
Title doc: remove obsolete --napi-modules doc entry (#64220)
⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch legendecas:cli-napi-modules -> nodejs:main
Labels node-api, author ready, needs-ci
Commits 1
- doc: remove obsolete --napi-modules doc entry
Committers 1
- Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: https://github.com/nodejs/node/pull/64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
--------------------------------------------------------------------------------
ℹ This PR was created on Tue, 30 Jun 2026 18:52:29 GMT
✔ Approvals: 4
✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/64220#pullrequestreview-4603771320
✔ - Adrian Estrada (@edsadr): https://github.com/nodejs/node/pull/64220#pullrequestreview-4603915973
✔ - Edy Silva (@geeksilva97): https://github.com/nodejs/node/pull/64220#pullrequestreview-4604828357
✔ - Harshitha K P (@HarshithaKP): https://github.com/nodejs/node/pull/64220#pullrequestreview-4610678742
✔ Last GitHub CI successful
ℹ Last Full PR CI on 2026-07-01T14:52:45Z: https://ci.nodejs.org/job/node-test-pull-request/74493/
- Querying data for job/node-test-pull-request/74493/
✔ Build data downloaded
✔ Last Jenkins CI successful
--------------------------------------------------------------------------------
✔ No git cherry-pick in progress
✔ No git am in progress
✔ No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/main up to date...
From https://github.com/nodejs/node
* branch main -> FETCH_HEAD
✔ origin/main is now up-to-date
- Downloading patch for 64220
From https://github.com/nodejs/node
* branch refs/pull/64220/merge -> FETCH_HEAD
✔ Fetched commits as b2e83150cb54..94da5035468e
--------------------------------------------------------------------------------
Auto-merging doc/api/cli.md
[main 41b4e7f163] doc: remove obsolete --napi-modules doc entry
Author: Chengzhong Wu <cwu631@bloomberg.net>
Date: Tue Jun 30 14:49:37 2026 -0400
4 files changed, 2 insertions(+), 15 deletions(-)
✔ Patches applied
--------------------------------------------------------------------------------
--------------------------------- New Message ----------------------------------
doc: remove obsolete --napi-modules doc entry

Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>

[main 1360e63644] doc: remove obsolete --napi-modules doc entry
Author: Chengzhong Wu <cwu631@bloomberg.net>
Date: Tue Jun 30 14:49:37 2026 -0400
4 files changed, 2 insertions(+), 15 deletions(-)
✔ 1360e6364443ed602d0e4dff94ebc33a6c0aae4f
✔ 0:0 no Assisted-by metadata assisted-by-is-trailer
✔ 0:0 no Co-authored-by metadata co-authored-by-is-trailer
✔ 0:0 skipping fixes-url fixes-url
✔ 0:0 blank line after title line-after-title
✔ 0:0 line-lengths are valid line-length
✔ 0:0 metadata is at end of message metadata-end
✔ 2:8 PR-URL is valid. pr-url
✔ 0:0 reviewers are valid reviewers
⚠ 1:0 "Signed-off-by" email does not match the commit author email signed-off-by
✔ 0:0 valid subsystems subsystem
✔ 0:0 Title is formatted correctly. title-format
✔ 0:0 Title is <= 50 columns. title-length

The following commits are landable on origin/main

  • 1360e63644 doc: remove obsolete --napi-modules doc entry

  • Removing temporary files
    ✔ Temporary files removed.
    To finish landing:
  1. Run:
    git push origin main
  2. Post "Landed in 1360e6364443" in doc: remove obsolete --napi-modules doc entry #64220
    gh pr comment doc: remove obsolete --napi-modules doc entry #64220 --body "Landed in 1360e6364443"
    gh pr close doc: remove obsolete --napi-modules doc entry #64220
https://github.com/nodejs/node/actions/runs/28615542785

@legendecaslegendecas added commit-queue PRs queued for automated landing through the Commit Queue. and removed commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. labels Jul 7, 2026
@nodejs-github-botnodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Jul 7, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 94164da into nodejs:mainJul 7, 2026
84 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 94164da

@github-project-automationgithub-project-automationBot moved this from In Progress to Done in Node-API Team ProjectJul 7, 2026
@legendecas
legendecas deleted the cli-napi-modules branch July 7, 2026 16:36
aduh95 pushed a commit that referenced this pull request Jul 21, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Jul 29, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
aduh95 pushed a commit that referenced this pull request Aug 6, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs with CI started, the required approvals, and no outstanding review comments.needs-ciPRs that need a full CI run.node-apiIssues and PRs related to Node-API.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants

@legendecas@nodejs-github-bot@edsadr@lpinca@geeksilva97@HarshithaKP
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content

doc: remove obsolete --napi-modules doc entry - #64220

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:cli-napi-modules
Jul 7, 2026
Merged

doc: remove obsolete --napi-modules doc entry#64220
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:cli-napi-modules

Conversation

@legendecas

Copy link
Copy Markdown
Member

--napi-modules has been a no-op for a long time. Remove it from the doc.

It is still kept as a no-op option.

@nodejs-github-botnodejs-github-bot added the needs-ci PRs that need a full CI run. label Jun 30, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
@legendecaslegendecas added the node-api Issues and PRs related to Node-API. label Jun 30, 2026
@legendecaslegendecas moved this from Need Triage to In Progress in Node-API Team ProjectJun 30, 2026
@geeksilva97geeksilva97 added author ready PRs with CI started, the required approvals, and no outstanding review comments. request-ci Add this label to start a Jenkins CI on a PR. labels Jun 30, 2026
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 1, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@legendecaslegendecas added the commit-queue PRs queued for automated landing through the Commit Queue. label Jul 2, 2026
@nodejs-github-botnodejs-github-bot added commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. and removed commit-queue PRs queued for automated landing through the Commit Queue. labels Jul 2, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/64220
✔ Done loading data for nodejs/node/pull/64220
----------------------------------- PR info ------------------------------------
Title doc: remove obsolete --napi-modules doc entry (#64220)
⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch legendecas:cli-napi-modules -> nodejs:main
Labels node-api, author ready, needs-ci
Commits 1
- doc: remove obsolete --napi-modules doc entry
Committers 1
- Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: https://github.com/nodejs/node/pull/64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
--------------------------------------------------------------------------------
ℹ This PR was created on Tue, 30 Jun 2026 18:52:29 GMT
✔ Approvals: 4
✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/64220#pullrequestreview-4603771320
✔ - Adrian Estrada (@edsadr): https://github.com/nodejs/node/pull/64220#pullrequestreview-4603915973
✔ - Edy Silva (@geeksilva97): https://github.com/nodejs/node/pull/64220#pullrequestreview-4604828357
✔ - Harshitha K P (@HarshithaKP): https://github.com/nodejs/node/pull/64220#pullrequestreview-4610678742
✔ Last GitHub CI successful
ℹ Last Full PR CI on 2026-07-01T14:52:45Z: https://ci.nodejs.org/job/node-test-pull-request/74493/
- Querying data for job/node-test-pull-request/74493/
✔ Build data downloaded
✔ Last Jenkins CI successful
--------------------------------------------------------------------------------
✔ No git cherry-pick in progress
✔ No git am in progress
✔ No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/main up to date...
From https://github.com/nodejs/node
* branch main -> FETCH_HEAD
✔ origin/main is now up-to-date
- Downloading patch for 64220
From https://github.com/nodejs/node
* branch refs/pull/64220/merge -> FETCH_HEAD
✔ Fetched commits as b2e83150cb54..94da5035468e
--------------------------------------------------------------------------------
Auto-merging doc/api/cli.md
[main 41b4e7f163] doc: remove obsolete --napi-modules doc entry
Author: Chengzhong Wu <cwu631@bloomberg.net>
Date: Tue Jun 30 14:49:37 2026 -0400
4 files changed, 2 insertions(+), 15 deletions(-)
✔ Patches applied
--------------------------------------------------------------------------------
--------------------------------- New Message ----------------------------------
doc: remove obsolete --napi-modules doc entry

Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>

[main 1360e63644] doc: remove obsolete --napi-modules doc entry
Author: Chengzhong Wu <cwu631@bloomberg.net>
Date: Tue Jun 30 14:49:37 2026 -0400
4 files changed, 2 insertions(+), 15 deletions(-)
✔ 1360e6364443ed602d0e4dff94ebc33a6c0aae4f
✔ 0:0 no Assisted-by metadata assisted-by-is-trailer
✔ 0:0 no Co-authored-by metadata co-authored-by-is-trailer
✔ 0:0 skipping fixes-url fixes-url
✔ 0:0 blank line after title line-after-title
✔ 0:0 line-lengths are valid line-length
✔ 0:0 metadata is at end of message metadata-end
✔ 2:8 PR-URL is valid. pr-url
✔ 0:0 reviewers are valid reviewers
⚠ 1:0 "Signed-off-by" email does not match the commit author email signed-off-by
✔ 0:0 valid subsystems subsystem
✔ 0:0 Title is formatted correctly. title-format
✔ 0:0 Title is <= 50 columns. title-length

The following commits are landable on origin/main

  • 1360e63644 doc: remove obsolete --napi-modules doc entry

  • Removing temporary files
    ✔ Temporary files removed.
    To finish landing:
  1. Run:
    git push origin main
  2. Post "Landed in 1360e6364443" in doc: remove obsolete --napi-modules doc entry #64220
    gh pr comment doc: remove obsolete --napi-modules doc entry #64220 --body "Landed in 1360e6364443"
    gh pr close doc: remove obsolete --napi-modules doc entry #64220
https://github.com/nodejs/node/actions/runs/28615542785

@legendecaslegendecas added commit-queue PRs queued for automated landing through the Commit Queue. and removed commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. labels Jul 7, 2026
@nodejs-github-botnodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Jul 7, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 94164da into nodejs:mainJul 7, 2026
84 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 94164da

@github-project-automationgithub-project-automationBot moved this from In Progress to Done in Node-API Team ProjectJul 7, 2026
@legendecas
legendecas deleted the cli-napi-modules branch July 7, 2026 16:36
aduh95 pushed a commit that referenced this pull request Jul 21, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
RafaelGSS pushed a commit that referenced this pull request Jul 29, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
aduh95 pushed a commit that referenced this pull request Aug 6, 2026
Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: #64220
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Adrian Estrada <edsadr@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Harshitha K P <harshitha014@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs with CI started, the required approvals, and no outstanding review comments.needs-ciPRs that need a full CI run.node-apiIssues and PRs related to Node-API.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants

@legendecas@nodejs-github-bot@edsadr@lpinca@geeksilva97@HarshithaKP