add a comment for argumentType.NumMethod() == 0

shogo82148 · shogo82148 · commit c2000973dd21 · 2022-12-22T10:57:35.000+09:00
diff --git a/lambda/handler.go b/lambda/handler.go
@@ -110,9 +110,12 @@ func handlerTakesContext(handler reflect.Type) (bool, error) {
 		if argumentType.Kind() != reflect.Interface {
 			return false, nil
 		}
+
+		// handlers like func(event any) are valid.
 		if argumentType.NumMethod() == 0 {
 			return false, nil
 		}
+
 		if !contextType.Implements(argumentType) || !argumentType.Implements(contextType) {
 			return false, fmt.Errorf("handler takes an interface, but it is not context.Context: %q", argumentType.Name())
 		}

Original file line number	Diff line number	Diff line change
`@@ -110,9 +110,12 @@ func handlerTakesContext(handler reflect.Type) (bool, error) {`
`110`	`110`	`if argumentType.Kind() != reflect.Interface {`
`111`	`111`	`return false, nil`
`112`	`112`	`}`
	`113`	`+`
	`114`	`+ // handlers like func(event any) are valid.`
`113`	`115`	`if argumentType.NumMethod() == 0 {`
`114`	`116`	`return false, nil`
`115`	`117`	`}`
	`118`	`+`
`116`	`119`	`if !contextType.Implements(argumentType) \|\| !argumentType.Implements(contextType) {`
`117`	`120`	`return false, fmt.Errorf("handler takes an interface, but it is not context.Context: %q", argumentType.Name())`
`118`	`121`	`}`