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

Adding warning about overwriting process #1308

Merged
merged 1 commit into from
Jun 20, 2017
Merged

Adding warning about overwriting process #1308

merged 1 commit into from
Jun 20, 2017

Conversation

zephraph
Copy link
Contributor

See these issues for references:

egoist/poi#193
https://github.com/chentsulin/electron-react-boilerplate/pull/245/files

Basically using something like the below example will break the process object compatibility that webpack includes. This just gives a warning about that.

new webpack.DefinePlugin({
   process: {
      env: {
         NODE_ENV: JSON.stringify('production')
      }
   }
})

See these issues for references:

egoist/poi#193
https://github.com/chentsulin/electron-react-boilerplate/pull/245/files

Basically using something like the below example will break the process object compatibility that webpack includes. This just gives a warning about that. 

```javascript
new webpack.DefinePlugin({
   process: {
      env: {
         NODE_ENV: JSON.stringify('production')
      }
   }
})
```
@simon04 simon04 merged commit 6ae2453 into webpack:master Jun 20, 2017
@simon04
Copy link
Collaborator

simon04 commented Jun 20, 2017

Thanks a lot for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants