Skip to content

tools: add resume-ci label handling - #65945

Open
panva wants to merge 2 commits into
nodejs:mainfrom
panva:resume-ci-label
Open

tools: add resume-ci label handling#65945
panva wants to merge 2 commits into
nodejs:mainfrom
panva:resume-ci-label

Conversation

@panva

@panva panva commented Sep 9, 2026

Copy link
Copy Markdown
Member

Adds [resume-ci] label handling alongside [request-ci]. Approved PRs are processed by the scheduled workflow using ncu-ci resume. Failed attempts receive [resume-ci-failed] and a comment with diagnostic output.

Both scripts refuse their operation when the other request label is present.

Before merging:

  • Merge nodejs/node-core-utils PR adding ncu-ci resume.
  • Publish an @node-core/utils release containing that change.
  • Create the [resume-ci] label in nodejs/node.
  • Create the [resume-ci-failed] label in nodejs/node.

Signed-off-by: Filip Skokan <panva.ip@gmail.com>
Assisted-by: Codex
@panva panva added the blocked PRs that are blocked by other issues or PRs. label Sep 9, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/actions
  • @nodejs/tsc

@nodejs-github-bot nodejs-github-bot added meta Issues and PRs related to the general management of the project. tools Issues and PRs related to the tools directory. labels Sep 9, 2026
@jasnell

jasnell commented Sep 9, 2026

Copy link
Copy Markdown
Member

I haven't yet looked into the details but a question: I assume that resume-ci will fail if new commits have been added since the last run?

@panva

panva commented Sep 9, 2026

Copy link
Copy Markdown
Member Author

I haven't yet looked into the details but a question: I assume that resume-ci will fail if new commits have been added since the last run?

yes.

--repo "$GITHUB_REPOSITORY" \
--label 'resume-ci' \
--json 'number' \
--search 'review:approved' \

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't yet looked into the details but a question: I assume that resume-ci will fail if new commits have been added since the last run?

No it won't be picked up until it's approved

@panva panva Sep 9, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, if an approval for the latest commit is there and [resume-ci] is still present then it will fail to start and report such.

Comment thread .github/workflows/auto-start-ci.yml Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

blocked PRs that are blocked by other issues or PRs. meta Issues and PRs related to the general management of the project. tools Issues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants