fix: Improve error handling

Author: robvanderleek

src/context.ts

@@ -1,12 +1,12 @@
-export function getRepoOwnerLogin(ctx: any) {
+export function getRepoOwnerLogin(ctx: any): string {
     return ctx.payload.repository.owner.login
 }
 
 export function getRepoOwnerId(ctx: any) {
     return ctx.payload.repository.owner.id
 }
 
-export function getRepoName(ctx: any) {
+export function getRepoName(ctx: any): string {
     return ctx.payload.repository.name
 }
 

src/webhooks/pull-request-closed.ts

@@ -18,14 +18,22 @@ export async function pullRequestClosed(_: Probot, ctx: Context<any>) {
                     issue_number: issueNumber
                 });
                 if (issueForBranch) {
-                    await ctx.octokit.issues.update({
-                        owner: owner,
-                        repo: repo,
-                        issue_number: issueNumber,
-                        state: 'closed'
-                    })
+                    await closeIssue(ctx, owner, repo, issueNumber);
                 }
             }
         }
     }
+}
+
+async function closeIssue(ctx: Context<any>, owner: string, repo: string, issueNumber: number) {
+    try {
+        await ctx.octokit.issues.update({
+            owner: owner,
+            repo: repo,
+            issue_number: issueNumber,
+            state: 'closed'
+        } as any)
+    } catch (e) {
+        return;
+    }
 }