- do not fail on missing issue number

- FIX #1178
mikepenz · Sep 13, 2024 · 568c6f5 · 568c6f5
1 parent 2e33034
commit 568c6f5
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 3 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/annotator.ts b/src/annotator.ts
@@ -230,6 +230,11 @@ export async function attachComment(
   detailsTable: SummaryTableRow[],
   flakySummary: SummaryTableRow[]
 ): Promise<void> {
+  if (!context.issue.number) {
+    core.warning(`⚠️ Action requires a valid issue number (PR reference) to be able to attach a comment..`)
+    return
+  }
+
   const identifier = buildCommentIdentifier(checkName)
 
   let comment = buildTable(table)

diff --git a/src/main.ts b/src/main.ts
@@ -155,7 +155,7 @@ export async function run(): Promise<void> {
 
     if (comment) {
       const octokit: InstanceType<typeof GitHub> = github.getOctokit(token)
-      attachComment(octokit, checkName, updateComment, table, detailTable, flakyTable)
+      await attachComment(octokit, checkName, updateComment, table, detailTable, flakyTable)
     }
 
     core.setOutput('summary', buildTable(table))