Merge pull request #33 from chrisgavin/safeexec-windows

Lookup path to `gh` safely on Windows.

Author: chrisgavin

cmd/root.go

@@ -20,6 +20,7 @@ import (
 	"github.com/chrisgavin/gh-dispatch/internal/workflow"
 	"github.com/cli/go-gh"
 	"github.com/cli/go-gh/pkg/repository"
+	"github.com/cli/safeexec"
 	"github.com/go-git/go-git/v5"
 	"github.com/pkg/errors"
 	log "github.com/sirupsen/logrus"
@@ -259,7 +260,12 @@ var rootCmd = &cobra.Command{
 				return err
 			}
 
-			command := exec.CommandContext(cmd.Context(), "gh", "run", "watch", "--repo", fmt.Sprintf("%s/%s/%s", currentRepository.Host(), currentRepository.Owner(), currentRepository.Name()), strconv.FormatInt(workflowRun.ID, 10))
+			ghPath, err := safeexec.LookPath("gh")
+			if err != nil {
+				return errors.Wrap(err, "Unable to find gh.")
+			}
+
+			command := exec.CommandContext(cmd.Context(), ghPath, "run", "watch", "--repo", fmt.Sprintf("%s/%s/%s", currentRepository.Host(), currentRepository.Owner(), currentRepository.Name()), strconv.FormatInt(workflowRun.ID, 10))
 			command.Stdout = os.Stdout
 			command.Stderr = os.Stderr
 			err = command.Run()

go.mod

@@ -5,6 +5,7 @@ go 1.17
 require (
 	github.com/AlecAivazis/survey/v2 v2.3.4
 	github.com/cli/go-gh v0.0.3
+	github.com/cli/safeexec v1.0.0
 	github.com/go-git/go-git/v5 v5.4.2
 	github.com/pkg/errors v0.9.1
 	github.com/sirupsen/logrus v1.8.1
@@ -17,7 +18,6 @@ require (
 	github.com/Microsoft/go-winio v0.4.16 // indirect
 	github.com/ProtonMail/go-crypto v0.0.0-20210428141323-04723f9f07d7 // indirect
 	github.com/acomagu/bufpipe v1.0.3 // indirect
-	github.com/cli/safeexec v1.0.0 // indirect
 	github.com/cli/shurcooL-graphql v0.0.1 // indirect
 	github.com/davecgh/go-spew v1.1.1 // indirect
 	github.com/emirpasic/gods v1.12.0 // indirect