Merge pull request #34 from kunalkushwaha/multiple-exclude-pathspec

vbatts · web-flow · commit 48acb1add096 · 2018-03-06T14:21:08.000-05:00
added support for multiple path in exclude list
diff --git a/git/commits.go b/git/commits.go
@@ -66,8 +66,11 @@ func Check(commit string) ([]byte, error) {
 		"--no-pager", "log", "--check",
 		fmt.Sprintf("%s^..%s", commit, commit),
 	}
-	if exclude := os.Getenv("GIT_CHECK_EXCLUDE"); exclude != "" {
-		args = append(args, "--", ".", fmt.Sprintf(":(exclude)%s", exclude))
+	if excludeEnvList := os.Getenv("GIT_CHECK_EXCLUDE"); excludeEnvList != "" {
+		excludeList := strings.Split(excludeEnvList, ":")
+		for _, exclude := range excludeList {
+			args = append(args, "--", ".", fmt.Sprintf(":(exclude)%s", exclude))
+		}
 	}
 	cmd := exec.Command("git", args...)
 	if debug() {

Original file line number	Diff line number	Diff line change
`@@ -66,8 +66,11 @@ func Check(commit string) ([]byte, error) {`
`66`	`66`	`"--no-pager", "log", "--check",`
`67`	`67`	`fmt.Sprintf("%s^..%s", commit, commit),`
`68`	`68`	`}`
`69`		`- if exclude := os.Getenv("GIT_CHECK_EXCLUDE"); exclude != "" {`
`70`		`- args = append(args, "--", ".", fmt.Sprintf(":(exclude)%s", exclude))`
	`69`	`+ if excludeEnvList := os.Getenv("GIT_CHECK_EXCLUDE"); excludeEnvList != "" {`
	`70`	`+ excludeList := strings.Split(excludeEnvList, ":")`
	`71`	`+ for _, exclude := range excludeList {`
	`72`	`+ args = append(args, "--", ".", fmt.Sprintf(":(exclude)%s", exclude))`
	`73`	`+ }`
`71`	`74`	`}`
`72`	`75`	`cmd := exec.Command("git", args...)`
`73`	`76`	`if debug() {`