Merge pull request cli#5357 from callumacrae/force-tty-pr-view

mislav · web-flow · commit 91c450ec604a · 2022-03-29T16:12:45.000+02:00
fix: respect `GH_FORCE_TTY` when running `gh pr view`
diff --git a/pkg/cmd/pr/view/view.go b/pkg/cmd/pr/view/view.go
@@ -98,7 +98,7 @@ func viewRun(opts *ViewOptions) error {
 		return err
 	}
 
-	connectedToTerminal := opts.IO.IsStdoutTTY() && opts.IO.IsStderrTTY()
+	connectedToTerminal := opts.IO.IsStdoutTTY()
 
 	if opts.BrowserMode {
 		openURL := pr.URL

Original file line number	Diff line number	Diff line change
`@@ -98,7 +98,7 @@ func viewRun(opts *ViewOptions) error {`
`98`	`98`	`return err`
`99`	`99`	`}`
`100`	`100`
`101`		`- connectedToTerminal := opts.IO.IsStdoutTTY() && opts.IO.IsStderrTTY()`
	`101`	`+ connectedToTerminal := opts.IO.IsStdoutTTY()`
`102`	`102`
`103`	`103`	`if opts.BrowserMode {`
`104`	`104`	`openURL := pr.URL`