Use PagerDuty/jsonlint to lint JSON files.

For whatever reason, `node -c` generally does not like JSON files.

Author: mparker17

README.md

@@ -44,6 +44,11 @@ the type of file(s) you want to lint:
 
             $ which node # returns a path if it is installed properly.
 
+    * If you want to syntax-lint JSON files, you will need to install
+        [jsonlint](https://github.com/PagerDuty/jsonlint#installation).
+
+            $ which jsonlint # returns a path if it is installed properly.
+
     * If you want to syntax-lint PHP files, you will need to
         [install PHP](http://php.net/manual/install.php).
 

pre-commit/1-js-syntax.sh

@@ -31,7 +31,7 @@ case "${1}" in
     ;;
 
   * )
-    for file in $(git diff-index --cached --name-only HEAD | grep -E '\.(js|json)') ; do
+    for file in $(git diff-index --cached --name-only HEAD | grep -E '\.(js)') ; do
       test_file "${file}"
     done
     ;;

pre-commit/1-json-syntax.sh

@@ -0,0 +1,38 @@
+#!/usr/bin/env bash
+
+# shellcheck disable=SC2034
+GREP_OPTIONS=""
+
+function test_file {
+  file="${1}"
+
+  if [ ! -f "${file}" ] ; then
+    return
+  fi
+
+  if which -s node ; then
+    echo "Running JSON syntax lint..."
+
+    # Set -e before and +e after for _required_ linters (i.e.: that will prevent
+    # commit, e.g.: syntax linters).
+    # Set +e before and -e after for _optional_ linters (i.e.: that will only
+    # output messages upon commit, e.g.: style linters).
+    set -e
+    jsonlint -c "$file"
+    set +e
+  else
+    echo "Can't run JSON syntax linter because the jsonlint executable isn't in the PATH."
+  fi
+}
+
+case "${1}" in
+  --about )
+    echo "JSON syntax lint."
+    ;;
+
+  * )
+    for file in $(git diff-index --cached --name-only HEAD | grep -E '\.(json)') ; do
+      test_file "${file}"
+    done
+    ;;
+esac