chore: use nesting instead of slice

Author: ldez

pkg/result/processors/sort_results.go

@@ -25,21 +25,21 @@ const (
 var _ Processor = (*SortResults)(nil)
 
 type SortResults struct {
-	cmps map[string][]comparator
+	cmps map[string]comparator
 
 	cfg *config.Output
 }
 
 func NewSortResults(cfg *config.Config) *SortResults {
 	return &SortResults{
-		cmps: map[string][]comparator{
+		cmps: map[string]comparator{
 			// For sorting we are comparing (in next order):
 			// file names, line numbers, position, and finally - giving up.
-			orderNameFile: {byFileName(), byLine(), byColumn()},
+			orderNameFile: byFileName().AddNext(byLine().AddNext(byColumn())),
 			// For sorting we are comparing: linter name
-			orderNameLinter: {byLinter()},
+			orderNameLinter: byLinter(),
 			// For sorting we are comparing: severity
-			orderNameSeverity: {bySeverity()},
+			orderNameSeverity: bySeverity(),
 		},
 		cfg: &cfg.Output,
 	}
@@ -58,7 +58,7 @@ func (sr SortResults) Process(issues []result.Issue) ([]result.Issue, error) {
 	var cmps []comparator
 	for _, name := range sr.cfg.SortOrder {
 		if c, ok := sr.cmps[name]; ok {
-			cmps = append(cmps, c...)
+			cmps = append(cmps, c)
 		} else {
 			return nil, fmt.Errorf("unsupported sort-order name %q", name)
 		}

pkg/result/processors/sort_results_test.go

@@ -254,6 +254,11 @@ func Test_mergeComparators(t *testing.T) {
 			cmps:     []comparator{bySeverity(), byLinter(), byFileName(), byLine(), byColumn()},
 			expected: "bySeverity > byLinter > byFileName > byLine > byColumn",
 		},
+		{
+			desc:     "nested",
+			cmps:     []comparator{bySeverity(), byLinter(), byFileName().AddNext(byLine().AddNext(byColumn()))},
+			expected: "bySeverity > byLinter > byFileName > byLine > byColumn",
+		},
 		{
 			desc:     "all reverse",
 			cmps:     []comparator{byColumn(), byLine(), byFileName(), byLinter(), bySeverity()},