ensure NODE_PATH environment variable is honored when checking for ex…

…traneous dependencies
facebook · Mar 10, 2017 · 00e0f58 · 00e0f58
1 parent 12108bd
commit 00e0f58
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 0 deletions.
diff --git a/packages/eslint-config-react-app/index.js b/packages/eslint-config-react-app/index.js
@@ -9,6 +9,8 @@
 
 'use strict';
 
+const resolveNodePath = require('./utils/resolveNodePath');
+
 // Inspired by https://github.com/airbnb/javascript but less opinionated.
 
 // We use eslint-loader so even warnings are very visible.
@@ -49,6 +51,7 @@ module.exports = {
     'import/resolver': {
       node: {
         extensions: ['.js', '.json'],
+        moduleDirectory: ['node_modules'].concat(resolveNodePath()),
       },
     },
   },

diff --git a/packages/eslint-config-react-app/package.json b/packages/eslint-config-react-app/package.json
@@ -8,6 +8,7 @@
     "url": "https://github.com/facebookincubator/create-react-app/issues"
   },
   "files": [
+    "utils",
     "index.js"
   ],
   "peerDependencies": {

diff --git a/packages/eslint-config-react-app/utils/resolveNodePath.js b/packages/eslint-config-react-app/utils/resolveNodePath.js
@@ -0,0 +1,11 @@
+'use strict';
+
+function resolveNodePath() {
+  const nodePaths = (process.env.NODE_PATH || '')
+    .split(process.platform === 'win32' ? ';' : ':')
+    .filter(Boolean);
+
+  return nodePaths;
+}
+
+module.exports = resolveNodePath;