Merge pull request #3259 from testwill/sort-string

mislav · web-flow · commit efa668511d66 · 2023-06-21T17:54:37.000+02:00
chore: use sort.Strings(...) instead of sort.Sort(sort.StringSlice(...))
diff --git a/github/client.go b/github/client.go
@@ -1320,7 +1320,7 @@ func (s scopeSet) String() string {
 	for scope := range s {
 		scopes = append(scopes, scope)
 	}
-	sort.Sort(sort.StringSlice(scopes))
+	sort.Strings(scopes)
 	return strings.Join(scopes, ", ")
 }
 

Original file line number	Diff line number	Diff line change
`@@ -1320,7 +1320,7 @@ func (s scopeSet) String() string {`
`1320`	`1320`	`for scope := range s {`
`1321`	`1321`	`scopes = append(scopes, scope)`
`1322`	`1322`	`}`
`1323`		`- sort.Sort(sort.StringSlice(scopes))`
	`1323`	`+ sort.Strings(scopes)`
`1324`	`1324`	`return strings.Join(scopes, ", ")`
`1325`	`1325`	`}`
`1326`	`1326`