pre-commit error

[zhangyuang]

[jandockx]

The example script is for GNU. macOS grep (BSD) does not support the -z / --null-data option (nor xargs -r). I could not find a complete workaround.

#!/usr/bin/env bash
#!/bin/bash

# `pre-commit` to ensure all JavaScript files staged for commit pass standard code style.
#
# Based on https://github.com/standard/standard#is-there-a-git-pre-commit-hook
# Tweaked
# - for Mac (where grep does not have the -z / --null-data option),
# - therefor, xargs-r does not work either
# - for .js files instead of .jsx files.

git diff --name-only --cached --relative | grep '\.js$' | xargs -t npx standard
if [[ $? -ne 0 ]]; then
  echo 'JavaScript Standard Style errors were detected. Aborting commit.'
  exit 1
fi

works on Mac,

• as long as there are no spaces in the paths to the staged js(x) files
• checks everything if there are no staged .js files

[feross]

Thanks for helping out, @jandockx!

One other workaround might be to install GNU grep using homebrew (a package manager for macOS).

brew install grep

This installs GNU grep as a binary called ggrep (note the extra leading g character). So, just change the references to grep to ggrep and you should be good :)