Skip to content

Commit

Permalink
Allow more neutral run
Browse files Browse the repository at this point in the history
  • Loading branch information
kachick committed Oct 5, 2023
1 parent 9e87e5f commit 93299c2
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion dist/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -12849,7 +12849,7 @@ async function getCheckRunSummaries(token, params) {
throw new Error("no runNodes");
}
return runNodes.map((run2) => ({
acceptable: run2.conclusion == "SUCCESS" || run2.conclusion === "SKIPPED" || checkSuite.conclusion === "SKIPPED",
acceptable: run2.conclusion == "SUCCESS" || run2.conclusion === "SKIPPED" || run2.conclusion === "NEUTRAL" && (checkSuite.conclusion === "SUCCESS" || checkSuite.conclusion === "SKIPPED"),
workflowPath: relative(`/${params.owner}/${params.repo}/actions/workflows/`, workflow.resourcePath),
checkSuiteStatus: checkSuite.status,
checkSuiteConclusion: checkSuite.conclusion,
Expand Down
4 changes: 3 additions & 1 deletion src/github-api.ts
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,9 @@ export async function getCheckRunSummaries(
}

return runNodes.map((run) => ({
acceptable: run.conclusion == 'SUCCESS' || run.conclusion === 'SKIPPED' || checkSuite.conclusion === 'SKIPPED',
acceptable: run.conclusion == 'SUCCESS' || run.conclusion === 'SKIPPED'
|| (run.conclusion === 'NEUTRAL'
&& (checkSuite.conclusion === 'SUCCESS' || checkSuite.conclusion === 'SKIPPED')),
workflowPath: relative(`/${params.owner}/${params.repo}/actions/workflows/`, workflow.resourcePath),

checkSuiteStatus: checkSuite.status,
Expand Down

0 comments on commit 93299c2

Please sign in to comment.