avoid nil pointer exception when we cant get digger config

motatoes · motatoes · commit d66a68ccdf7e · 2024-10-31T07:46:18.000Z
diff --git a/backend/controllers/github.go b/backend/controllers/github.go
@@ -378,7 +378,6 @@ func handlePullRequestEvent(gh utils.GithubClientProvider, payload *github.PullR
 
 	diggerYmlStr, ghService, config, projectsGraph, _, _, err := getDiggerConfigForPR(gh, installationId, repoFullName, repoOwner, repoName, cloneURL, prNumber)
 	if err != nil {
-		utils.InitCommentReporter(ghService, prNumber, fmt.Sprintf(":x: Could not load digger config, error: %v", err))
 		log.Printf("getDiggerConfigForPR error: %v", err)
 		return fmt.Errorf("error getting digger config")
 	}

Original file line number	Diff line number	Diff line change
`@@ -378,7 +378,6 @@ func handlePullRequestEvent(gh utils.GithubClientProvider, payload *github.PullR`
`378`	`378`
`379`	`379`	`diggerYmlStr, ghService, config, projectsGraph, _, _, err := getDiggerConfigForPR(gh, installationId, repoFullName, repoOwner, repoName, cloneURL, prNumber)`
`380`	`380`	`if err != nil {`
`381`		`- utils.InitCommentReporter(ghService, prNumber, fmt.Sprintf(":x: Could not load digger config, error: %v", err))`
`382`	`381`	`log.Printf("getDiggerConfigForPR error: %v", err)`
`383`	`382`	`return fmt.Errorf("error getting digger config")`
`384`	`383`	`}`