Merge pull request #1604 from patheticpat/fix-vi-regex

Fix recognition logic for vi editor variants again
mislav · Oct 17, 2017 · 5ce945e · 5ce945e
2 parents 14f4cb8 + ad61211
commit 5ce945e
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/github/editor.go b/github/editor.go
@@ -111,8 +111,8 @@ func (e *Editor) readContent() (content []byte, err error) {
 
 func openTextEditor(program, file string) error {
 	editCmd := cmd.New(program)
-	r := regexp.MustCompile(`\b(?:[gm]?vim|vi)\b`)
-	if r.MatchString(program) {
+	r := regexp.MustCompile(`\b(?:[gm]?vim|vi)(?:\.exe)?$`)
+	if r.MatchString(editCmd.Name) {
 		editCmd.WithArg("--cmd")
 		editCmd.WithArg("set ft=gitcommit tw=0 wrap lbr")
 	}