From 75b71505c882e9707cf81eb618bad170aae42c82 Mon Sep 17 00:00:00 2001 From: "Babak K. Shandiz" Date: Fri, 24 Apr 2026 22:19:59 +0100 Subject: [PATCH] refactor(discussion view): use slices package for label sorting Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- pkg/cmd/discussion/view/view.go | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/pkg/cmd/discussion/view/view.go b/pkg/cmd/discussion/view/view.go index 8cb027b44..ca94b238e 100644 --- a/pkg/cmd/discussion/view/view.go +++ b/pkg/cmd/discussion/view/view.go @@ -3,7 +3,7 @@ package view import ( "fmt" "io" - "sort" + "slices" "strings" "time" @@ -331,12 +331,13 @@ func labelList(labels []client.DiscussionLabel, cs *iostreams.ColorScheme) strin return "" } - sort.SliceStable(labels, func(i, j int) bool { - return strings.ToLower(labels[i].Name) < strings.ToLower(labels[j].Name) + sortedLabels := slices.Clone(labels) + slices.SortStableFunc(sortedLabels, func(i, j client.DiscussionLabel) int { + return strings.Compare(i.Name, j.Name) }) - names := make([]string, len(labels)) - for i, l := range labels { + names := make([]string, len(sortedLabels)) + for i, l := range sortedLabels { if cs == nil { names[i] = l.Name } else {