-
Notifications
You must be signed in to change notification settings - Fork 98
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
Fix webpack environment variables #2120
Conversation
Signed-off-by: Karoly Albert Szabo <[email protected]>
Signed-off-by: Karoly Albert Szabo <[email protected]>
Codecov Report
@@ Coverage Diff @@
## develop #2120 +/- ##
========================================
Coverage 95.31% 95.31%
========================================
Files 110 110
Lines 2581 2581
Branches 104 104
========================================
Hits 2460 2460
Misses 112 112
Partials 9 9 |
1 similar comment
Codecov Report
@@ Coverage Diff @@
## develop #2120 +/- ##
========================================
Coverage 95.31% 95.31%
========================================
Files 110 110
Lines 2581 2581
Branches 104 104
========================================
Hits 2460 2460
Misses 112 112
Partials 9 9 |
webpack.renderer.config.js
Outdated
NODE_ENV: JSON.stringify(process.env.NODE_ENV) | ||
} | ||
}), | ||
new webpack.EnvironmentPlugin([`NODE_ENV`, `RELEASE`, `STARGATE`, `RPC`, `RELEASE`, `FAUCET`]), |
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.
new webpack.EnvironmentPlugin([`NODE_ENV`, `RELEASE`, `STARGATE`, `RPC`, `RELEASE`, `FAUCET`]), | |
new webpack.EnvironmentPlugin([`NODE_ENV`, `STARGATE`, `RPC`, `RELEASE`, `FAUCET`]), |
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.
Thanks! I think I will fall back on the dear old DefinePlugin, MacOS and linux behave slightly differently (or can be just different types of shells) with this one, it seems to just forward
- Different behaviour for falsey values like empty strings on different OS
Description:
Thank you! 🚀
For contributor:
CHANGELOG.md
with issue # and GitHub usernameFiles changed
in the github PR explorerFor reviewer: