docs(pr/shared): add a TODO on decoupling PR finder from IO

babakks · babakks · commit 6f698407fb03 · 2025-09-18T10:25:49.000+01:00
Signed-off-by: Babak K. Shandiz &lt;babakks@github.com&gt;
diff --git a/pkg/cmd/pr/shared/finder.go b/pkg/cmd/pr/shared/finder.go
@@ -198,6 +198,7 @@ func (f *finder) Find(opts FindOptions) (*api.PullRequest, ghrepo.Interface, err
 		return nil, nil, err
 	}
 
+	// TODO: Decouple the PR finder from IO
 	// TODO(josebalius): Should we be guarding here?
 	if !opts.DisableProgress && f.progress != nil {
 		f.progress.StartProgressIndicator()

Original file line number	Diff line number	Diff line change
`@@ -198,6 +198,7 @@ func (f finder) Find(opts FindOptions) (api.PullRequest, ghrepo.Interface, err`
`198`	`198`	`return nil, nil, err`
`199`	`199`	`}`
`200`	`200`
	`201`	`+ // TODO: Decouple the PR finder from IO`
`201`	`202`	`// TODO(josebalius): Should we be guarding here?`
`202`	`203`	`if !opts.DisableProgress && f.progress != nil {`
`203`	`204`	`f.progress.StartProgressIndicator()`