Skip to content

doc: remove obsolete cctest node.gyp instructions - #64814

Open
soulee-dev wants to merge 1 commit into
nodejs:mainfrom
soulee-dev:doc-cctest-sources
Open

doc: remove obsolete cctest node.gyp instructions#64814
soulee-dev wants to merge 1 commit into
nodejs:mainfrom
soulee-dev:doc-cctest-sources

Conversation

@soulee-dev

Copy link
Copy Markdown
Contributor

Since 6aa42f9, configure.py collects every .cc and .h file under test/cctest into the node_cctest_sources variable, so new C++ unit tests no longer need to be listed in the sources of the cctest target.

Document the feature-gated source lists that still need to be updated by hand.

Refs: #56885

Since 6aa42f9, configure.py collects every .cc and .h file under
test/cctest into the node_cctest_sources variable, so a new C++ unit
test no longer has to be listed in the sources of the cctest target.
Document the feature-gated source lists that still need to be updated
by hand instead.

Refs: nodejs#56885

Signed-off-by: Soul Lee <alus20x@gmail.com>
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/tsc

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Jul 29, 2026

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@mcollina mcollina added the commit-queue Add this label to land a pull request using GitHub Actions. label Jul 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

commit-queue Add this label to land a pull request using GitHub Actions. doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants