Skip to content

Commit 281b265

Browse files
committed
Address review comments
1 parent 129d771 commit 281b265

File tree

4 files changed

+5
-4
lines changed

4 files changed

+5
-4
lines changed

lib/init-action-post.js

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/init-action-post-helper.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -374,6 +374,7 @@ test("saves overlay status when overlay-base analysis did not complete successfu
374374
attemptedToBuildOverlayBaseDatabase: true,
375375
builtOverlayBaseDatabase: false,
376376
job: {
377+
checkRunId: undefined,
377378
workflowRunId: 12345,
378379
workflowRunAttempt: 1,
379380
name: "analyze",

src/init-action-post-helper.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -283,7 +283,7 @@ async function recordOverlayStatus(
283283
attemptedToBuildOverlayBaseDatabase: true,
284284
builtOverlayBaseDatabase: false,
285285
},
286-
Number.isNaN(checkRunId) ? undefined : checkRunId,
286+
checkRunId !== undefined && checkRunId >= 0 ? checkRunId : undefined,
287287
);
288288

289289
const diskUsage = await checkDiskUsage(logger);

src/overlay/status.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ export function createOverlayStatus(
7474
workflowRunId: getWorkflowRunID(),
7575
workflowRunAttempt: getWorkflowRunAttempt(),
7676
name: getRequiredEnvParam("GITHUB_JOB"),
77-
...(checkRunId !== undefined && { checkRunId }),
77+
checkRunId,
7878
};
7979
return {
8080
...attributes,

0 commit comments

Comments
 (0)