modify validators to check json an format, stop on any error

jq_all_the_things.sh

@@ -1,17 +1,28 @@
 #!/bin/bash
 
+# Seeds sponge, from moreutils
+
+#Validate all Jsons first
+for dir in `find . -name "*.json"`
+do
+  echo validating ${dir}
+  cat ${dir} | jq . >/dev/null
+  rc=$?
+  if [[ $rc != 0 ]]; then exit $rc; fi
+done
+
 set -e
 set -x
 
-# Seeds sponge, from moreutils
-
 for dir in clusters/*.json
 do
+    # Beautify it
     cat ${dir} | jq . | sponge ${dir}
 done
 
 for dir in galaxies/*.json
 do
+    # Beautify it
     cat ${dir} | jq . | sponge ${dir}
 done
 

validate_all.sh

@@ -1,21 +1,34 @@
 #!/bin/bash
 
+# Check Jsons format, and beautify
+./jq_all_the_things.sh
+rc=$?
+if [[ $rc != 0 ]]; then 
+    exit $rc
+fi
+
 set -e
 set -x
 
-./jq_all_the_things.sh
-
-diffs=`git status --porcelain | wc -l`
-
-if ! [ $diffs -eq 0 ]; then
-	echo "Please make sure you run ./jq_all_the_things.sh before commiting."
-	exit 1
-fi
+# fixme to remove.. 
+# Not need anymore ow, jq stop upon error...
+# diffs=`git status --porcelain | wc -l`
+#
+#if ! [ $diffs -eq 0 ]; then
+#	echo "Please make sure you run ./jq_all_the_things.sh before commiting."
+#	exit
+#fi
 
+# Validate schemas
 for dir in clusters/*.json
 do
   echo -n "${dir}: "
   jsonschema -i ${dir} schema_clusters.json
+  rc=$?
+  if [[ $rc != 0 ]]; then 
+    echo "Error on ${dir}"
+    exit $rc
+  fi
   echo ''
 done
 
@@ -23,6 +36,10 @@ for dir in galaxies/*.json
 do
   echo -n "${dir}: "
   jsonschema -i ${dir} schema_galaxies.json
+  rc=$?
+  if [[ $rc != 0 ]]; then 
+    echo "Error on ${dir}"
+    exit $rc
+  fi
   echo ''
 done
-