Updated jshint-pre-commit-hook. Improved formatting and created grunt…

… task to copy hook file to the ".git/hooks" folder.

.jshint-pre-commit-hook

@@ -0,0 +1,36 @@
+#!/bin/sh
+#
+# Put this file into ".git\hooks" folder
+# It would jshint-validate scripts only from the current commit
+#
+
+files=$(git diff --cached --name-only --diff-filter=ACM | grep ".js$")
+if [ "$files" = "" ]; then
+    exit 0
+fi
+
+valid=true
+
+printf '\n%s\n' "---- Validating JavaScript with JSHint: -----"
+
+for file in ${files}; do
+
+    result=$(jshint ${file} --config .jshintrc)
+
+    if [ "$result" == "" ]; then
+        printf '\n%s' "-- JSHint Passed: ${file}"
+    else
+        printf '\n\n%s' "-- JSHint Failed: ${file}"
+		printf '\n%s\n' "$result"
+        valid=false
+    fi
+done
+
+printf '\n\n%s\n' "---- JavaScript validation complete.    -----"
+
+if ${valid}; then
+    printf '\n%s\n' echo "Commit Succeeded"
+else
+    printf '\n%s\n' "Commit Failed: Please fix all above JSHint errors and try again"
+    exit 1
+fi

Gruntfile.js

@@ -26,17 +26,25 @@ module.exports = function(grunt) {
             options: {
                 jshintrc: true
             }
+        },
+        copy: {
+            "jshint-pre-commit-hook": {
+                src: '.jshint-pre-commit-hook',
+                dest: '.git/hooks/pre-commit'
+            }
         }
     });
 
     grunt.loadNpmTasks('grunt-git');
     grunt.loadNpmTasks('grunt-text-replace');
     grunt.loadNpmTasks('grunt-shell');
     grunt.loadNpmTasks('grunt-contrib-jshint');
+    grunt.loadNpmTasks('grunt-contrib-copy');
 
     grunt.registerTask('default', function() {
         grunt.task.run('replace', 'shell:gitAdd', 'shell:gitCommit');
     });
 
     grunt.registerTask('validatejs', ['jshint']);
+    grunt.registerTask('jshint-hook', ['copy:jshint-pre-commit-hook']);
 };

app/chapter-23/products.js

@@ -48,7 +48,7 @@ angular.module('exampleApp', ['increment', 'ngResource', 'ngRoute', 'ngAnimate']
     }
 
     $scope.cancelEdit = function () {
-        $location.path("/list");
+        $location.path('/list');
     };
 
     $scope.updateProduct = function (product) {

package.json

@@ -8,10 +8,13 @@
   },
   "devDependencies": {
     "grunt": "^0.4.5",
+    "grunt-contrib-copy": "^0.7.0",
     "grunt-contrib-jshint": "^0.10.0",
     "grunt-git": "~0.2.14",
     "grunt-shell": "~1.1.1",
     "grunt-text-replace": "~0.3.12",
-    "jshint": "^2.5.10"
+    "jshint": "^2.5.10",
+    "pre-commit": "0.0.11",
+    "which": "^1.0.8"
   }
 }