nodejs / nodejs/node-core-utils
Cancelled GitHub CI rerun after success is treated as failure
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 314
- Forks
- 137
- Avg merge
- 2d 8h
- Merged PRs (30d)
- 24
Description
PR: https://github.com/nodejs/node/pull/43085
Action rerun attempt: https://github.com/nodejs/node/actions/runs/2320332585/attempts/7
git-node land output: https://github.com/nodejs/node/pull/43085#issuecomment-1126858386
At the moment of landing, the latest run of that action was cancelled, there were 5 prior successful runs, and GitHub UI reported everything as green.
However, ncu treated it as Last GitHub CI failed error, which is either wrong (not sure if easily fixable, as it requires failproof checking previous attempts) or confusing (Last GitHub CI cancelled might be a better error message).
Details: in PRChecker.checkGitHubCI(), commit.checkSuites.nodes[2] for this case was
{
app: { slug: 'github-actions' },
conclusion: 'CANCELLED',
status: 'COMPLETED'
}
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in PRChecker.checkGitHubCI() and inspect how commit.checkSuites.nodes is interpreted, using the linked PR, action attempt, and git-node land output as reproduction context. Verify behavior when the latest GitHub Actions run is cancelled after earlier successful runs. Done means the reported CI state and error message accurately reflect that case, with tests covering it.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- cli
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100