all: clean up few minor things
Change-Id: Icfc780e06f9cadbb74ec9252bc67eea0e70f7bb9
Reviewed-on: https://go-review.googlesource.com/c/pkgsite-metrics/+/522915
Reviewed-by: Maceo Thompson <maceothompson@google.com>
Run-TryBot: Zvonimir Pavlinovic <zpavlinovic@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
diff --git a/internal/govulncheck/govulncheck.go b/internal/govulncheck/govulncheck.go
index 2d12126..5e53422 100644
--- a/internal/govulncheck/govulncheck.go
+++ b/internal/govulncheck/govulncheck.go
@@ -167,7 +167,7 @@
func (v1 *WorkVersion) Equal(v2 *WorkVersion) bool {
if v1 == nil || v2 == nil {
- return v1 == v2
+ return false
}
return v1.GoVersion == v2.GoVersion &&
v1.WorkerVersion == v2.WorkerVersion &&
diff --git a/internal/worker/govulncheck_scan.go b/internal/worker/govulncheck_scan.go
index ab24ae6..7545e05 100644
--- a/internal/worker/govulncheck_scan.go
+++ b/internal/worker/govulncheck_scan.go
@@ -122,10 +122,10 @@
// unrecoverableError returns true iff errorCategory encodes that
// the project has an error that is unrecoverable from the perspective
-// of govulncheck. Examples are build issues and the lack of go.mod file.
+// of govulncheck. One example is build issues.
func unrecoverableError(errorCategory string) bool {
switch errorCategory {
- case derrors.CategorizeError(derrors.LoadPackagesError): // We model build usses as a general load error.
+ case derrors.CategorizeError(derrors.LoadPackagesError): // We model build issues as a general load error.
return true
default:
return false
