-
-
Notifications
You must be signed in to change notification settings - Fork 3.3k
docs(config/node): add boolean
to the type
#3050
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
Conversation
Follow up for #1317.
Preview is ready Built with commit c327877 https://deploy-preview-3050--webpackjsorg-netlify.netlify.com |
@@ -16,7 +16,7 @@ This feature is provided by webpack's internal [`NodeStuffPlugin`](https://githu | |||
|
|||
## `node` | |||
|
|||
`object` | |||
`boolean | object` | |||
|
|||
This is an object where each property is the name of a Node global or module and each value may be one of the following... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update this too since it can accept bool :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The documentation of the false
value is at the end:
Since webpack 3.0.0, the
node
option may be set tofalse
to completely turn off theNodeStuffPlugin
andNodeSourcePlugin
plugins.
Would you like me to put that up front instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, missed that one, this looks good! ✨
Thanks |
Follow up for #1317.
describe your changes...