Merge pull request #100 from tetsuharuohzeki/fix2

Workaround to fail to parse json in pull request sync path

json.go

@@ -6,8 +6,6 @@ import (
 	"os"
 
 	"encoding/json"
-
-	"github.com/google/go-github/v32/github"
 )
 
 // PushEventData is workaround to use *github.PushEvent.
@@ -55,9 +53,8 @@ func loadJSONFileForPushEventData(path string) *PushEventData {
 // If we available, we should use *github.PushEvent.
 // But we cannot use it by parsing error. See https://github.com/cats-oss/github-action-detect-unmergeable/issues/71
 type PullRequestEventData struct {
-	Action      *string             `json:"action,omitempty"`
-	Number      *int                `json:"number,omitempty"`
-	PullRequest *github.PullRequest `json:"pull_request,omitempty"`
+	Action *string `json:"action,omitempty"`
+	Number *int    `json:"number,omitempty"`
 }
 
 func (p *PullRequestEventData) GetAction() string {

main.go

@@ -151,15 +151,19 @@ func onPullRequestEvent(githubClient *github.Client, githubEventPath string, rep
 		return
 	}
 
-	pr := eventData.PullRequest
-	prNumber := pr.GetNumber()
+	prNumber := eventData.GetNumber()
 	if prNumber == 0 {
-		log.Fatalln("we cannot get the PR number for this pull request")
+		log.Println("we cannot get the PR number for this pull request")
 		return
 	}
 
-	if _, isUnmergeable := checkWhetherThisPullRequestNeedRebase(pr, needRebaseLabel); isUnmergeable {
-		log.Printf("#%v is not mergeable", prNumber)
+	_, isUnmergeable, err := shouldMarkPullRequestNeedRebase(githubClient, repoOwner, repoName, prNumber, needRebaseLabel)
+	if err != nil {
+		return
+	}
+
+	if isUnmergeable {
+		log.Printf("#%v is not mergeable. We don't do anything", prNumber)
 		return
 	}
 

operation.go

@@ -158,6 +158,7 @@ func shouldMarkPullRequestNeedRebase(client *github.Client, owner, repo string,
 	ctx := context.Background()
 	newPRInfoResponse, _, err := client.PullRequests.Get(ctx, owner, repo, number)
 	if err != nil {
+		log.Printf("Getting info about #%v was failed.", number)
 		return
 	}