Skip to content

[ci] Skip CLA check in the merge queue - #2471

Merged
andreaskurth merged 1 commit into
lowRISC:masterfrom
andreaskurth:cla-merge-queue
Aug 11, 2026
Merged

[ci] Skip CLA check in the merge queue#2471
andreaskurth merged 1 commit into
lowRISC:masterfrom
andreaskurth:cla-merge-queue

Conversation

@andreaskurth

Copy link
Copy Markdown
Contributor

Prior to this commit, the CLA Assistant workflow was not triggered
for merge groups. The required CLAAssistant check therefore never
reported a result for a merge group, which blocked the merge queue
until the check timed out.

The CLA Assistant action cannot check a merge group, because it needs
a pull request to determine the authors and to post its comment on.

This commit adds a merge_group trigger to the workflow. The
condition on the only step of the workflow is false for that event, so
the job completes without running any step and the CLAAssistant
check reports success. The CLA is thus only checked on a pull
request, before that pull request enters the merge queue.

Prior to this commit, the `CLA Assistant` workflow was not triggered
for merge groups.  The required `CLAAssistant` check therefore never
reported a result for a merge group, which blocked the merge queue
until the check timed out.

The CLA Assistant action cannot check a merge group, because it needs
a pull request to determine the authors and to post its comment on.

This commit adds a `merge_group` trigger to the workflow.  The
condition on the only step of the workflow is false for that event, so
the job completes without running any step and the `CLAAssistant`
check reports success.  The CLA is thus only checked on a pull
request, before that pull request enters the merge queue.

Signed-off-by: Andreas Kurth <adk@lowrisc.org>
@github-actions

github-actions Bot commented Aug 11, 2026

Copy link
Copy Markdown

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@andreaskurth

Copy link
Copy Markdown
Contributor Author

I have read the CLA Document. By submitting this pull request comment, I am hereby confirming my acceptance of the terms of the CLA Document and my agreement to be legally bound by its terms.

@andreaskurth
andreaskurth enabled auto-merge August 11, 2026 14:40
@andreaskurth
andreaskurth added this pull request to the merge queue Aug 11, 2026
Merged via the queue into lowRISC:master with commit 7b5df75 Aug 11, 2026
13 of 14 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 11, 2026
@andreaskurth
andreaskurth deleted the cla-merge-queue branch August 11, 2026 16:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants