lint: add sqlclosecheck (#8129)

Picking up from #7709

.golangci.yml

@@ -9,6 +9,7 @@ linters:
     - ineffassign
     - misspell
     - nolintlint
+    - sqlclosecheck
     - typecheck
     - unconvert
     - unparam

cmd/contact-auditor/main.go

@@ -131,7 +131,7 @@ func (c contactAuditor) run(ctx context.Context, resChan chan *result) error {
 		}
 	}
 	// Ensure the query wasn't interrupted before it could complete.
-	err = rows.Close()
+	err = rows.Close() //nolint:sqlclosecheck // the lint wants us to do this in a defer instead, but we want to return the error
 	if err != nil {
 		return err
 	} else {

test/db.go

@@ -113,6 +113,7 @@ func allTableNamesInDB(ctx context.Context, db CleanUpDB) ([]string, error) {
 	if err != nil {
 		return nil, err
 	}
+	defer r.Close()
 	var ts []string
 	for r.Next() {
 		tableName := ""