Merge branch 'master' into markdownlint

Author: sebastien-boulle

README.md

@@ -285,18 +285,19 @@ you can add this to your configuration so commit messages are checked locally:
 Into `.pre-commit-config.yaml`:
 
 ```yaml
-
----
+default_stages: [commit]
 repos:
----
-- repo: https://github.com/lumapps/commit-message-validator
-  rev: master
-  hooks:
-    - id: commit-message-validator
-      stages: [commit-msg]
-      args: [--allow-temp]
+  - repo: https://github.com/lumapps/commit-message-validator
+    rev: master
+    hooks:
+      - id: commit-message-validator
+        stages: [commit-msg]
+        args: [--allow-temp]
 ```
 
+`default_stages` tells which stage to install hooks that do not specify
+a `stages` option.
+
 Then run `pre-commit install --hook-type commit-msg` to install the
 `commit-message-validator`
 

check_message.sh

@@ -31,7 +31,8 @@ then
   exit
 fi
 
-MESSAGE=$(<"$1")
+# removing comment lines from message
+MESSAGE=$(sed '/^#/d' "$1")
 
 FIRST_WORD=${MESSAGE%% *}
 if [[ "${FIRST_WORD,,}" == merge ]]
@@ -44,6 +45,6 @@ fi
 # print message so you don't lose it in case of errors
 # (in case you are not using `-m` option)
 echo "Options: JIRA=$COMMIT_VALIDATOR_NO_JIRA, TEMP=$COMMIT_VALIDATOR_ALLOW_TEMP"
-printf "checking commit message:\n\n#BEGIN#\n%s\n#END#\n\n" "$(grep -v "#" <<< "$MESSAGE")"
+printf "checking commit message:\n\n#BEGIN#\n%s\n#END#\n\n" "$MESSAGE"
 
 validate "$MESSAGE"