Avoid re-wrapping of "could not determine suitable remote" error

mislav · mislav · commit 135efda6d325 · 2019-11-14T19:31:33.000+01:00
diff --git a/command/pr_create.go b/command/pr_create.go
@@ -37,7 +37,7 @@ func prCreate(cmd *cobra.Command, _ []string) error {
 
 	remote, err := guessRemote(ctx)
 	if err != nil {
-		return errors.Wrap(err, "could not determine suitable remote")
+		return err
 	}
 
 	if err = git.Push(remote, fmt.Sprintf("HEAD:%s", head)); err != nil {
@@ -179,7 +179,7 @@ func prCreate(cmd *cobra.Command, _ []string) error {
 func guessRemote(ctx context.Context) (string, error) {
 	remotes, err := ctx.Remotes()
 	if err != nil {
-		return "", errors.Wrap(err, "could not determine suitable remote")
+		return "", errors.Wrap(err, "could not read git remotes")
 	}
 
 	// TODO: consolidate logic with fsContext.BaseRepo

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ func prCreate(cmd *cobra.Command, _ []string) error {`
`37`	`37`
`38`	`38`	`remote, err := guessRemote(ctx)`
`39`	`39`	`if err != nil {`
`40`		`- return errors.Wrap(err, "could not determine suitable remote")`
	`40`	`+ return err`
`41`	`41`	`}`
`42`	`42`
`43`	`43`	`if err = git.Push(remote, fmt.Sprintf("HEAD:%s", head)); err != nil {`
`@@ -179,7 +179,7 @@ func prCreate(cmd *cobra.Command, _ []string) error {`
`179`	`179`	`func guessRemote(ctx context.Context) (string, error) {`
`180`	`180`	`remotes, err := ctx.Remotes()`
`181`	`181`	`if err != nil {`
`182`		`- return "", errors.Wrap(err, "could not determine suitable remote")`
	`182`	`+ return "", errors.Wrap(err, "could not read git remotes")`
`183`	`183`	`}`
`184`	`184`
`185`	`185`	`// TODO: consolidate logic with fsContext.BaseRepo`