Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: mention that node options are space-separated #14709

Conversation

gabrielschulhof
Copy link
Contributor

The documentation does not mention that the value of NODE_OPTIONS is a
space-separated list.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

doc

@nodejs-github-bot nodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label Aug 9, 2017
The documentation does not mention that the value of NODE_OPTIONS is a
space-separated list.
@gabrielschulhof gabrielschulhof force-pushed the node-options-are-space-separated branch from 445c1ee to cff4124 Compare August 9, 2017 07:30
@mscdex mscdex added the doc Issues and PRs related to the documentations. label Aug 9, 2017
@vsemozhetbyt vsemozhetbyt added the cli Issues and PRs related to the Node.js command line interface. label Aug 9, 2017
@XadillaX
Copy link
Contributor

Copy link
Contributor

@XadillaX XadillaX left a comment

Choose a reason for hiding this comment

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

LGTM

@gabrielschulhof gabrielschulhof self-assigned this Aug 10, 2017
@gabrielschulhof
Copy link
Contributor Author

@gabrielschulhof
Copy link
Contributor Author

Landed in 6fe72c8.

gabrielschulhof pushed a commit that referenced this pull request Aug 13, 2017
The documentation does not mention that the value of NODE_OPTIONS is a
space-separated list.

PR-URL: #14709
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Khaidi Chu <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
addaleax pushed a commit that referenced this pull request Aug 13, 2017
The documentation does not mention that the value of NODE_OPTIONS is a
space-separated list.

PR-URL: #14709
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Khaidi Chu <[email protected]>
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
@addaleax addaleax mentioned this pull request Aug 13, 2017
@gabrielschulhof gabrielschulhof deleted the node-options-are-space-separated branch September 5, 2017 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++ Issues and PRs that require attention from people who are familiar with C++. cli Issues and PRs related to the Node.js command line interface. doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants