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

refactor(discussion list): encapsulate printing within printDiscussions

Browse source 
Signed-off-by: Babak K. Shandiz <babakks@github.com>
This commit is contained in:
Babak K. Shandiz 2026-04-14 15:03:02 +01:00
parent d45ce940eb
commit 8d70cc9ca8
No known key found for this signature in database
GPG key ID: 9472CAEFF56C742E
1 changed files with 8 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

17
pkg/cmd/discussion/list/list.go
View file

@ -228,13 +228,7 @@ func listRun(opts *ListOptions) error {
fmt.Fprintf(opts.IO.ErrOut, "failed to start pager: %v\n", err)
}
isTerminal := opts.IO.IsStdoutTTY()
if isTerminal {
title := listHeader(ghrepo.FullName(repo), len(result.Discussions), result.TotalCount, opts.State)
fmt.Fprintf(opts.IO.Out, "\n%s\n\n", title)
}
printDiscussions(opts, result.Discussions, result.TotalCount)
printDiscussions(opts, ghrepo.FullName(repo), result.Discussions, result.TotalCount)
return nil
}
@ -294,11 +288,16 @@ func listHeader(repoName string, count, total int, state string) string {
}
}
func printDiscussions(opts *ListOptions, discussions []client.Discussion, totalCount int) {
func printDiscussions(opts *ListOptions, repoName string, discussions []client.Discussion, totalCount int) {
isTerminal := opts.IO.IsStdoutTTY()
cs := opts.IO.ColorScheme()
now := opts.Now()
if isTerminal {
title := listHeader(repoName, len(discussions), totalCount, opts.State)
fmt.Fprintf(opts.IO.Out, "\n%s\n\n", title)
}
headers := []string{"ID", "TITLE", "CATEGORY", "LABELS", "ANSWERED", "UPDATED"}
if !isTerminal {
headers = []string{"ID", "STATE", "TITLE", "CATEGORY", "LABELS", "ANSWERED", "UPDATED"}
@ -309,7 +308,7 @@ func printDiscussions(opts *ListOptions, discussions []client.Discussion, totalC
if isTerminal {
idColor := cs.Green
if d.Closed {
idColor = cs.Gray
idColor = cs.Muted
}
tp.AddField(fmt.Sprintf("#%d", d.Number), tableprinter.WithColor(idColor))
} else {