Skip to content
Closed
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
doc: add useful v8 option section
Fixed a typo addition in doc/cli.md
Fixes: #32252
  • Loading branch information
nimit95 committed Mar 14, 2020
commit 5209db887b6485d995ce30a57f1d5134d1ed5c65
4 changes: 1 addition & 3 deletions doc/api/cli.md
Original file line number Diff line number Diff line change
Expand Up @@ -1180,8 +1180,6 @@ V8 options that are allowed are:
`--perf-basic-prof-only-functions`, `--perf-basic-prof`,
`--perf-prof-unwinding-info`, and `--perf-prof` are only available on Linux.

### Useful v8 Options
##
### `NODE_PATH=path[:…]`
<!-- YAML
added: v0.1.32
Expand Down Expand Up @@ -1387,7 +1385,7 @@ threadpool by setting the `'UV_THREADPOOL_SIZE'` environment variable to a value
greater than `4` (its current default value). For more information, see the
[libuv threadpool documentation][].

## Useful v8 Options
## Useful v8 options
Comment thread
nimit95 marked this conversation as resolved.
Outdated

These are v8 options and fall outside Node.js' responsibility.
Comment thread
nimit95 marked this conversation as resolved.
Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
These are v8 options and fall outside Node.js' responsibility.
V8 has CLI options. Any V8 CLI option that is provided to `node` will be passed on to V8 to handle. V8's options have _no stability guarantee_. The V8 team themselves don't consider them to be part of their formal API, and reserve the right to change them at any time. The Node.js team does not consider them covered by the Node.js stability guarantees. Many of the V8 options are of interest only to V8 developers. Despite this, there is a small set of V8 options that are widely applicable to Node.js, and they are documented here.

I think we should say something like the above.


Expand Down