Skip to content

IOT_DS-8566: fix CLA trigger condition and enable CLA workflow triggers#62

Open
Ziegler András (AndrasZiegler) wants to merge 1 commit intomasterfrom
codex/IOT_DS-8566-fix-cla-trigger
Open

IOT_DS-8566: fix CLA trigger condition and enable CLA workflow triggers#62
Ziegler András (AndrasZiegler) wants to merge 1 commit intomasterfrom
codex/IOT_DS-8566-fix-cla-trigger

Conversation

@AndrasZiegler
Copy link
Contributor

@AndrasZiegler Ziegler András (AndrasZiegler) commented Feb 25, 2026

This PR updates the CLA Assistant workflow trigger condition and enables trigger events for CLA processing.

Changes:

  • Replace the CLA step condition with a guarded expression that handles pull_request_target and qualifying issue_comment events
  • Enable issue_comment and pull_request_target triggers when they were commented out

Ticket: IOT_DS-8566


Note

Low Risk
Workflow-only change that affects when the CLA check runs; low code risk but could impact PR gating if the condition is incorrect.

Overview
Updates the 01-CLA-Assistant GitHub Actions workflow to run the CLA check job only for pull_request_target events or for PR issue_comment events that either request recheck or include the CLA signing text (and guards against non-PR comments).

This moves the conditional from the "CLA Assistant" step to a job-level if, while keeping the existing triggers enabled for issue_comment: created and pull_request_target lifecycle events.

Written by Cursor Bugbot for commit 6f61ce3. This will update automatically on new commits. Configure here.

@AndrasZiegler Ziegler András (AndrasZiegler) force-pushed the codex/IOT_DS-8566-fix-cla-trigger branch 4 times, most recently from 5c35d0a to f0cf440 Compare February 25, 2026 15:14
@AndrasZiegler Ziegler András (AndrasZiegler) marked this pull request as ready for review February 26, 2026 07:00
@sonarqubecloud
Copy link

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant