go/analysis/passes/slog: add check for slog.Group

slog.Group now takes a ...any, so add it to the list
of functions to check.

Change-Id: I73025158a6cf55c7a6689a36f3c2c58df6d0db3f
Reviewed-on: https://go-review.googlesource.com/c/tools/+/494098
Run-TryBot: Jonathan Amsterdam <[email protected]>
gopls-CI: kokoro <[email protected]>
Reviewed-by: Tim King <[email protected]>
TryBot-Result: Gopher Robot <[email protected]>

Author: jba

go/analysis/passes/slog/slog.go

@@ -174,10 +174,9 @@ func kvFuncSkipArgs(fn *types.Func) (int, bool) {
 	}
 	recv := fn.Type().(*types.Signature).Recv()
 	if recv == nil {
-		// TODO: If #59204 is accepted, uncomment the lines below.
-		// if fn.Name() == "Group" {
-		// 	return 0, true
-		// }
+		if fn.Name() == "Group" {
+			return 0, true
+		}
 		skip, ok := slogOutputFuncs[fn.Name()]
 		return skip, ok
 	}

go/analysis/passes/slog/testdata/src/a/a.go

@@ -143,6 +143,8 @@ func All() {
 
 	r.Add(1, 2) // want `slog.Record.Add arg "1" should be a string or a slog.Attr`
 
+	_ = slog.Group("a", 1, 2, 3) // want `slog.Group arg "2" should be a string or a slog.Attr`
+
 }
 
 // Used in tests by package b.