Git pre-commit hook failing in GitHub for mac (works on command line)

I've created a very simple pre-commit script:-

#!/usr/bin/env sh

# Run tests
npm test
if [ $? -ne 0 ]; then
  echo "Tests failed. Aborting.."
  exit 1
fi

exit 0

When I run git commit at the command line (with failing tests), I get the expected exit 1 with the message Tests failed. Aborting...

However, If I use GitHub for Mac however I get:

.git/hooks/pre-commit: line 5: npm: command not found
Tests failed. Aborting..
 (256)

I'm guessing its down to npm not being available to the execution environment that GitHub for Mac is using, but I've been tearing my hair out trying to work out how to fix this.

Answers


Resolved. As globally installed node modules end up under /usr/local/bin I simply needed to add the following at the beginning of my pre-commit:

PATH=$PATH:/usr/local/bin:/usr/local/sbin

i.e. appending both /usr/local/bin and /usr/local/sbin to PATH at the point of execution.


The $PATH variable that is available in GUI environments like Github Desktop and Sourcetree is different than the one available in the terminal. By default the $PATH available in the GUI environments can't find your node modules. As was stated in the previous answer, you can ensure that /usr/local/bin is in the path by adding

PATH=$PATH:/usr/local/bin:/usr/local/sbin

In my case this did not work because I am using Node Version Manager, which stores different versions of Node and makes it easy to upgrade and switch Node versions. It stores your node_modules for each version of Node in a separate file. Here is the code I used to get around this problem:

#!/usr/bin/env bash

PATH="/usr/local/bin:$PATH"

if [ -f $HOME/.nvm/nvm.sh ]
then
  . $HOME/.nvm/nvm.sh
  PATH="$HOME/.nvm/versions/node/$(nvm current)/bin:$PATH"
fi

This checks for NVM, and if it exists, loads it and uses it to find the path to the node modules for the currently used version of Node. If you are only trying to access node modules and don't need to get at anything special you can skip adding the sbin folder, e.g. /usr/local/sbin


For me it was that I didn't have a dependency installed that was being called from pre-commit. In my case this was composer, so a brew install composer helped me out.

For future peeps, check if you are missing any dependancies being called in your pre-commit file by opening your-project-directory/.git/hooks/pre-commit in your favorite text editor and install missing the dependancies as needed.


Need Your Help

How to create NSCollectionView programmatically from scratch?

xcode cocoa nscollectionview

NSCollectionView remains one of the most mysterious parts of the Cocoa API that I've ever seen. Documentation is poor and there are many moving parts, many of which are often implemented in Interf...

How can I return a 404/50x status code from a Grails Controller?

grails error-handling groovy http-status-code-404

I have a controller that needs to return a 404 page and status code on certain conditions. I can't seem to find out how to do this in Grails. A coworker recommended this method: