Skip to content

Commit a689d8f

Browse files
Sergey VilgelmSergey Vilgelm
Sergey Vilgelm
authored and
Sergey Vilgelm
committed
print with colors the Message and code
1 parent 08315e7 commit a689d8f

File tree

3 files changed

+17
-17
lines changed

3 files changed

+17
-17
lines changed

Diff for: pkg/golinters/goanalysis/linter.go

+1-13
Original file line numberDiff line numberDiff line change
@@ -195,22 +195,10 @@ func buildIssues(diags []Diagnostic, linterNameBuilder func(diag *Diagnostic) st
195195
text = fmt.Sprintf("%s: %s", diag.Analyzer.Name, diag.Message)
196196
}
197197

198-
var suggestedFixes string
199-
if len(diag.SuggestedFixes) > 0 {
200-
elems := []string{}
201-
for _, fix := range diag.SuggestedFixes {
202-
elems = append(elems, fix.Message)
203-
for _, text := range fix.TextEdits {
204-
elems = append(elems, string(text.NewText))
205-
}
206-
}
207-
suggestedFixes = strings.Join(elems, "\n")
208-
}
209-
210198
issues = append(issues, result.Issue{
211199
FromLinter: linterName,
212200
Text: text,
213-
SuggestedFixes: suggestedFixes,
201+
SuggestedFixes: diag.SuggestedFixes,
214202
Pos: diag.Position,
215203
Pkg: diag.Pkg,
216204
})

Diff for: pkg/printers/text.go

+14-3
Original file line numberDiff line numberDiff line change
@@ -66,9 +66,20 @@ func (p Text) printIssue(i *result.Issue) {
6666
}
6767

6868
func (p Text) printSuggestedFixes(i *result.Issue) {
69-
suggestedFixes := strings.TrimSpace(i.SuggestedFixes)
70-
if suggestedFixes != "" {
71-
fmt.Fprintln(logutils.StdOut, suggestedFixes)
69+
var text string
70+
if len(i.SuggestedFixes) > 0 {
71+
for _, fix := range i.SuggestedFixes {
72+
text += p.SprintfColored(color.FgRed, "%s\n", strings.TrimSpace(fix.Message))
73+
elems := []string{}
74+
for _, text := range fix.TextEdits {
75+
elems = append(elems, strings.TrimSpace(string(text.NewText)))
76+
}
77+
text += strings.Join(elems, "\n") + "\n"
78+
}
79+
}
80+
81+
if text != "" {
82+
fmt.Fprintln(logutils.StdOut, text)
7283
}
7384
}
7485

Diff for: pkg/result/issue.go

+2-1
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import (
55
"fmt"
66
"go/token"
77

8+
"golang.org/x/tools/go/analysis"
89
"golang.org/x/tools/go/packages"
910
)
1011

@@ -27,7 +28,7 @@ type InlineFix struct {
2728
type Issue struct {
2829
FromLinter string
2930
Text string
30-
SuggestedFixes string
31+
SuggestedFixes []analysis.SuggestedFix
3132

3233
Severity string
3334

0 commit comments

Comments
 (0)