return error if no patch details

owenrumney · Jan 21, 2022 · 1300224 · 1300224
1 parent 00a96c0
commit 1300224
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/commenter/commenter.go b/commenter/commenter.go
@@ -195,7 +195,7 @@ func buildComment(file, comment string, line int, info commitFileInfo) *github.P
 func getCommitInfo(file *github.CommitFile) (*commitFileInfo, error) {
 	patch := file.GetPatch()
 	if patch == "" {
-		return nil, nil
+		return nil, fmt.Errorf("ignoring [%s] because it has no patch informations, probably a binary?", *file.Filename)
 	}
 
 	groups := patchRegex.FindAllStringSubmatch(file.GetPatch(), -1)