internal/scan: print the summary even when there are no findings

ianthehat · julieqiu · commit 2dd8d104bb82 · 2023-07-13T04:28:52.000Z
Fixes golang/go#61328 Change-Id: Ie1099cf1e55665d82b40da7c91a3edf6c55f6ab6 Reviewed-on: https://go-review.googlesource.com/c/vuln/+/508920 Reviewed-by: Julie Qiu <julieqiu@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> Run-TryBot: Ian Cottrell <iancottrell@google.com> Auto-Submit: Ian Cottrell <iancottrell@google.com>
diff --git a/internal/scan/text.go b/internal/scan/text.go
@@ -69,9 +69,6 @@ func Flush(h govulncheck.Handler) error {
 }
 
 func (h *TextHandler) Flush() error {
-	if len(h.findings) == 0 {
-		return nil
-	}
 	fixupFindings(h.osvs, h.findings)
 	h.byVulnerability(h.findings)
 	h.summary(h.findings)

Original file line number	Diff line number	Diff line change
`@@ -69,9 +69,6 @@ func Flush(h govulncheck.Handler) error {`
`69`	`69`	`}`
`70`	`70`
`71`	`71`	`func (h *TextHandler) Flush() error {`
`72`		`- if len(h.findings) == 0 {`
`73`		`- return nil`
`74`		`- }`
`75`	`72`	`fixupFindings(h.osvs, h.findings)`
`76`	`73`	`h.byVulnerability(h.findings)`
`77`	`74`	`h.summary(h.findings)`