Support fixing multiple empty lines

bombsimon · bombsimon · commit e88f18246bc9 · 2023-10-03T22:32:35.000+02:00
diff --git a/pkg/golinters/whitespace.go b/pkg/golinters/whitespace.go
@@ -49,8 +49,8 @@ func NewWhitespace(settings *config.WhitespaceSettings) *goanalysis.Linter {
 				switch issue.MessageType {
 				case whitespace.MessageTypeRemove:
 					report.LineRange = &result.Range{
-						From: issue.LineNumber,
-						To:   issue.LineNumber,
+						From: issue.LineNumbers[0],
+						To:   issue.LineNumbers[len(issue.LineNumbers)-1],
 					}
 
 					report.Replacement = &result.Replacement{
diff --git a/test/testdata/fix/out/whitespace.go b/test/testdata/fix/out/whitespace.go
@@ -41,7 +41,6 @@ func oneLeadingNewlineWithCommentFunc() {
 }
 
 func twoLeadingNewlines() {
-
 	fmt.Println("Hello world")
 }
 

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,6 @@ func oneLeadingNewlineWithCommentFunc() {`
`41`	`41`	`}`
`42`	`42`
`43`	`43`	`func twoLeadingNewlines() {`
`44`		`-`
`45`	`44`	`fmt.Println("Hello world")`
`46`	`45`	`}`
`47`	`46`