Address review comments

Author: owen-mc

go/ql/lib/ext/github.com.masterminds.squirrel.model.yml

@@ -80,3 +80,5 @@ extensions:
       - ["group:squirrel", "UpdateBuilder", True, "Suffix", "", "", "Argument[0]", "sql-injection", "manual"]
       - ["group:squirrel", "UpdateBuilder", True, "Table", "", "", "Argument[0]", "sql-injection", "manual"]
       # UpdateBuilder.Where has to be modeled in QL to avoid FPs when a non-string argument is used
+
+    # There are summary models for Row.Scan, RowScanner.Scan, {Insert,Delete,Select,Update}Builder.Scan and {Insert,Delete,Select,Update}Builder.ScanContext modeled in QL

go/ql/lib/semmle/go/frameworks/Squirrel.qll

@@ -54,7 +54,7 @@ module Squirrel {
     FunctionOutput outp;
 
     BuilderScan() {
-      // signature: func (b InsertBuilder) Scan(dest ...interface{}) error
+      // signature: func (b {Insert,Delete,Select,Update}Builder) Scan(dest ...interface{}) error
       this.hasQualifiedName(packagePath(),
         ["DeleteBuilder", "InsertBuilder", "SelectBuilder", "UpdateBuilder"], "Scan") and
       inp.isReceiver() and
@@ -71,7 +71,7 @@ module Squirrel {
     FunctionOutput outp;
 
     BuilderScanContext() {
-      // signature: func (b InsertBuilder) ScanContext(ctx context.Context, dest ...interface{}) error
+      // signature: func (b {Insert,Delete,Select,Update}Builder) ScanContext(ctx context.Context, dest ...interface{}) error
       this.hasQualifiedName(packagePath(),
         ["DeleteBuilder", "InsertBuilder", "SelectBuilder", "UpdateBuilder"], "ScanContext") and
       inp.isReceiver() and