Skip to content

Commit 6b89230

Browse files
committed
fix: NoPos is a begining of a file
1 parent c3e693d commit 6b89230

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed

pkg/result/processors/fixer.go

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ import (
1010
"errors"
1111
"fmt"
1212
"go/format"
13-
"go/token"
1413
"os"
1514
"slices"
1615

@@ -77,7 +76,8 @@ func (p Fixer) process(issues []result.Issue) ([]result.Issue, error) {
7776
for i := range issues {
7877
issue := issues[i]
7978

80-
if issue.SuggestedFixes == nil || skipTextEditWithoutPosition(&issue) {
79+
if issue.SuggestedFixes == nil || skipNoTextEdit(&issue) {
80+
println("Fixing", len(issues), "issues")
8181
notFixableIssues = append(notFixableIssues, issue)
8282
continue
8383
}
@@ -197,16 +197,14 @@ func (p Fixer) printStat() {
197197
p.sw.PrintStages()
198198
}
199199

200-
func skipTextEditWithoutPosition(issue *result.Issue) bool {
200+
func skipNoTextEdit(issue *result.Issue) bool {
201201
var onlyMessage int
202202
var count int
203203
for _, sf := range issue.SuggestedFixes {
204-
for _, edit := range sf.TextEdits {
205-
count++
206-
if edit.Pos == token.NoPos && edit.End == token.NoPos {
207-
onlyMessage++
208-
}
204+
if len(sf.TextEdits) == 0 {
205+
onlyMessage++
209206
}
207+
count++
210208
}
211209

212210
return count == onlyMessage

0 commit comments

Comments
 (0)