Linting in pre-commit hook #10483

Not everything can be fixed automatically by eslint, but at least we
avoid pushing broken things to CI.

bin/pre-commit.sh

@@ -15,6 +15,19 @@ if [ "$files" != "" ]; then
     echo "$files" | xargs git add
 fi
 
+files=$(git diff --cached --name-only --diff-filter=ACMR | grep -E 'client/.*\.(html|ts)$')
+if [ "$files" != "" ]; then
+
+    # Run eslint before commit
+    printf -- '--lint-file-patterns %s\n' $files | xargs ./node_modules/.bin/ng lint --fix
+    if [ $? -ne 0 ]; then
+        pass=false
+    fi
+
+    # Automatically add files that may have been fixed by eslint
+    echo "$files" | xargs git add
+fi
+
 files=$(git diff --cached --name-only --diff-filter=ACMR | grep -E '\.(php|phtml)$')
 if [ "$files" != "" ]; then