From 968a912a07aa6bf32c70aaba84902ca9bc7baf40 Mon Sep 17 00:00:00 2001 From: Kynan Ware <47394200+BagToad@users.noreply.github.com> Date: Mon, 26 Jan 2026 15:04:32 -0700 Subject: [PATCH] Remove outdated TODO comments in survey.go Cleaned up obsolete TODO comments related to assignee and reviewer selection logic in the MetadataSurvey function. --- pkg/cmd/pr/shared/survey.go | 4 ---- 1 file changed, 4 deletions(-) diff --git a/pkg/cmd/pr/shared/survey.go b/pkg/cmd/pr/shared/survey.go index 9ba600ae4..e350671b9 100644 --- a/pkg/cmd/pr/shared/survey.go +++ b/pkg/cmd/pr/shared/survey.go @@ -209,7 +209,6 @@ func MetadataSurvey(p Prompt, io *iostreams.IOStreams, baseRepo ghrepo.Interface // Populate the list of selectable assignees and their default selections. // This logic maps the default assignees from `state` to the corresponding actors or users // so that the correct display names are preselected in the prompt. - // TODO: KW21 This will need to go away since we're going to dynamically load assignees via search. var assignees []string var assigneesDefault []string if state.ActorAssignees { @@ -267,9 +266,6 @@ func MetadataSurvey(p Prompt, io *iostreams.IOStreams, baseRepo ghrepo.Interface fmt.Fprintln(io.ErrOut, "warning: no available reviewers") } } - // TODO: KW21 This will need to change to use MultiSelectWithSearch once it's implemented. - // MultiSelectWithSearch will return the selected strings directly instead of indices, - // so the logic here will need to be updated accordingly. if isChosen("Assignees") { if len(assignees) > 0 { selected, err := p.MultiSelect("Assignees", assigneesDefault, assignees)