Skip to content

tools: add fallback cleanup job for CQ - #65005

Open
aduh95 wants to merge 1 commit into
nodejs:mainfrom
aduh95:cq-fallback
Open

tools: add fallback cleanup job for CQ#65005
aduh95 wants to merge 1 commit into
nodejs:mainfrom
aduh95:cq-fallback

Conversation

@aduh95

@aduh95 aduh95 commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

In case of cancellation (e.g. timeout) occuring while the CQ was dealing with a PR, we likely want the PR back into the queue.

Refs: #64972

In case of cancellation (e.g. timeout) occuring while the CQ was dealing
with a PR, we likely want the PR back into the queue.

Signed-off-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/actions

@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Aug 4, 2026
done
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
PRs: ${{ needs.get_mergeable_prs.outputs.numbers }}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Rather than re-adding the label, why don't we change the CQ to not remove it until it either completes or fails (reducing our calls to the GitHub API)

concurrency: ${{ github.workflow }}

This line would prevent a PR from being picked up twice, so I see no harm in waiting to remove the label

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The consequence would be there would be merged PRs with the label, but yeah I'm not sure if that's actually an issue. Feel free to push to this branch or open an alternative PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

meta Issues and PRs related to the general management of the project.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants