Removed NODE_PATH from env.js

env.js is only for variables injected into the app.
facebook · Sep 2, 2016 · 6e94bd8 · 6e94bd8
1 parent afe25df
commit 6e94bd8
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 5 deletions.
diff --git a/config/env.js b/config/env.js
@@ -14,7 +14,6 @@
 
 var REACT_APP = /^REACT_APP_/i;
 var NODE_ENV = JSON.stringify(process.env.NODE_ENV || 'development');
-var NODE_PATH = process.env.NODE_PATH || '';
 
 module.exports = Object
   .keys(process.env)
@@ -23,6 +22,5 @@ module.exports = Object
     env['process.env.' + key] = JSON.stringify(process.env[key]);
     return env;
   }, {
-    'process.env.NODE_ENV': NODE_ENV,
-    'process.env.NODE_PATH': NODE_PATH
+    'process.env.NODE_ENV': NODE_ENV
   });
diff --git a/config/webpack.config.dev.js b/config/webpack.config.dev.js
@@ -69,7 +69,7 @@ module.exports = {
   resolve: {
     // This allows you to set a root for where webpack should look for modules.
     // This enables you to use absolute imports from the root.
-    root: path.resolve(env['process.env.NODE_PATH']),
+    root: path.resolve(process.env.NODE_PATH || ''),
     // These are the reasonable defaults supported by the Node ecosystem.
     extensions: ['.js', '.json', ''],
     alias: {

diff --git a/config/webpack.config.prod.js b/config/webpack.config.prod.js
@@ -64,7 +64,7 @@ module.exports = {
   resolve: {
     // This allows you to set a root for where webpack should look for modules.
     // This enables you to use absolute imports from the root.
-    root: path.resolve(env['process.env.NODE_PATH']),
+    root: path.resolve(process.env.NODE_PATH || ''),
     // These are the reasonable defaults supported by the Node ecosystem.
     extensions: ['.js', '.json', ''],
     alias: {