Merge pull request #77 from peter-evans/dev

(fix) handle case when pull request body is null
peter-evans · Sep 2, 2020 · 7dd4ee1 · 7dd4ee1
2 parents 762c957 + 7c2e432
commit 7dd4ee1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -1560,7 +1560,9 @@ function run() {
             if (isPullRequest) {
                 const pullRequest = yield githubHelper.getPull(github.context.repo, github.context.payload.issue.number);
                 // Truncate the body to keep the size of the payload under the max
-                pullRequest.body = pullRequest.body.slice(0, 1000);
+                if (pullRequest.body) {
+                    pullRequest.body = pullRequest.body.slice(0, 1000);
+                }
                 clientPayload['pull_request'] = pullRequest;
             }
             // Dispatch for each matching configuration

diff --git a/src/main.ts b/src/main.ts
@@ -164,7 +164,9 @@ async function run(): Promise<void> {
         github.context.payload.issue.number
       )
       // Truncate the body to keep the size of the payload under the max
-      pullRequest.body = pullRequest.body.slice(0, 1000)
+      if (pullRequest.body) {
+        pullRequest.body = pullRequest.body.slice(0, 1000)
+      }
       clientPayload['pull_request'] = pullRequest
     }