STACKITGit/cli
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Remove outdated TODO comments in survey.go

Browse source 
Cleaned up obsolete TODO comments related to assignee and reviewer selection logic in the MetadataSurvey function.
This commit is contained in:
Kynan Ware 2026-01-26 15:04:32 -07:00
parent a8053d6e93
commit 968a912a07
1 changed files with 0 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

4
pkg/cmd/pr/shared/survey.go
View file

@ -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)