Skip to content

Add a workflow to automatically flag PRs without a signed CLA#2627

Merged
GCHQDeveloper581 merged 2 commits into
gchq:masterfrom
GCHQDeveloper581:feature/automatically-flag-prs-without-signed-cla
Jul 3, 2026
Merged

Add a workflow to automatically flag PRs without a signed CLA#2627
GCHQDeveloper581 merged 2 commits into
gchq:masterfrom
GCHQDeveloper581:feature/automatically-flag-prs-without-signed-cla

Conversation

@GCHQDeveloper581

Copy link
Copy Markdown
Contributor

Description
Add a workflow that runs when a PR is created/reopened/resynced, or commented up that checks for any CLA Assistant comments and sets/clears the "awaiting cla" label according to the CLA status recorder in such comments

Existing Issue
This is the first half of the solution to #2626

AI disclosure
Initial coding by Claude Opus 4.8

Subsequently checked and neatened by hand. I understand, and accept responsibility for, all the resulting code.

Test Coverage
Automated testing not really possible. Will need some careful supervision in early operation.

* use commit hash to select action
* refine bot name
* refine comment identification regexs
* use AWAITING_LABEL env variable in a more conventional way
@GCHQDeveloper581

Copy link
Copy Markdown
Contributor Author

Note: because this only triggers when the PR is created (etc) or commented on (*) there will need to be some out-of-band action (whether fully manual, or scripted) to ensure the "awaiting cla" flag is set correctly on legacy PRs.

(*) and in the steady-state I think this is the correct (most efficient) way to do it.

@GCHQDeveloper581 GCHQDeveloper581 merged commit 740d350 into gchq:master Jul 3, 2026
2 checks passed
@GCHQDeveloper581 GCHQDeveloper581 deleted the feature/automatically-flag-prs-without-signed-cla branch July 3, 2026 12:13
@GCHQDeveloper581

Copy link
Copy Markdown
Contributor Author

A script has now been run on the repo to check that all (and only) open PRs with unsigned CLA do have the "awaiting cla" label set.

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.

2 participants