SC2069 The order of the 2>&1 and the redirect matters.

The 2>&1 has to be last. http://www.shellcheck.net/ [ci skip]
tko · Feb 8, 2015 · a60ff5d · a60ff5d
1 parent 62977ea
commit a60ff5d
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/tools/prepare-commit-msg b/tools/prepare-commit-msg
@@ -11,7 +11,7 @@ add_ci_skip ()
   source="$1"
 
   # Don't add [ci skip] if it's already in the commit message source
-  grep '\[ci skip\]' "$source" 2>&1 >/dev/null
+  grep '\[ci skip\]' "$source" >/dev/null 2>&1
   [ $? -eq 0 ] && return
 
   if [ $(git diff --cached --name-only | grep --count "$pattern") -eq 0 ]; then