robojerk/codeql-action

Fix bug where run attempt was reported as run ID (#1661)

Browse source
This commit is contained in:
Angela P Wen 2023-04-27 11:05:34 -07:00 • committed by GitHub
parent dcf71cf79b
commit 8b12d99ee5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
lib/workflow.js generated
View file

@ -267,7 +267,7 @@ exports.getWorkflowRunID = getWorkflowRunID;
* Get the workflow run attempt number. * Get the workflow run attempt number.
*/ */
function getWorkflowRunAttempt() { function getWorkflowRunAttempt() {
const workflowRunAttemptString = (0, util_1.getRequiredEnvParam)("GITHUB_RUN_ID"); const workflowRunAttemptString = (0, util_1.getRequiredEnvParam)("GITHUB_RUN_ATTEMPT");
const workflowRunAttempt = parseInt(workflowRunAttemptString, 10); const workflowRunAttempt = parseInt(workflowRunAttemptString, 10);
if (Number.isNaN(workflowRunAttempt)) { if (Number.isNaN(workflowRunAttempt)) {
throw new Error(`GITHUB_RUN_ATTEMPT must define a non NaN workflow run attempt. Current value is ${workflowRunAttemptString}`); throw new Error(`GITHUB_RUN_ATTEMPT must define a non NaN workflow run attempt. Current value is ${workflowRunAttemptString}`);

2
lib/workflow.js.map
View file

File diff suppressed because one or more lines are too long

2
src/workflow.ts
View file

@ -331,7 +331,7 @@ export function getWorkflowRunID(): number {
* Get the workflow run attempt number. * Get the workflow run attempt number.
*/ */
export function getWorkflowRunAttempt(): number { export function getWorkflowRunAttempt(): number {
const workflowRunAttemptString = getRequiredEnvParam("GITHUB_RUN_ID"); const workflowRunAttemptString = getRequiredEnvParam("GITHUB_RUN_ATTEMPT");
const workflowRunAttempt = parseInt(workflowRunAttemptString, 10); const workflowRunAttempt = parseInt(workflowRunAttemptString, 10);
if (Number.isNaN(workflowRunAttempt)) { if (Number.isNaN(workflowRunAttempt)) {
throw new Error( throw new Error(