Merge pull request #578 from Lucas-C/master

asottile · web-flow · commit f909deb037da · 2017-08-04T12:01:17.000-04:00
NodeJS hooks compatibilty fix for Cygwin - fix #455
diff --git a/pre_commit/languages/node.py b/pre_commit/languages/node.py
@@ -17,10 +17,11 @@
 
 
 def get_env_patch(venv):  # pragma: windows no cover
+    config = os.path.join(venv, 'bin') if sys.platform == 'cygwin' else venv
     return (
         ('NODE_VIRTUAL_ENV', venv),
-        ('NPM_CONFIG_PREFIX', venv),
-        ('npm_config_prefix', venv),
+        ('NPM_CONFIG_PREFIX', config),
+        ('npm_config_prefix', config),
         ('NODE_PATH', os.path.join(venv, 'lib', 'node_modules')),
         ('PATH', (os.path.join(venv, 'bin'), os.pathsep, Var('PATH'))),
     )
