Skip to content

fix(ci): skip the CLA comment when nothing is missing - #174

Merged
ericwang401 merged 1 commit into
mainfrom
fix/cla-quiet-on-pass
Oct 16, 2026
Merged

ericwang401 merged 1 commit into
mainfrom
fix/cla-quiet-on-pass

Conversation

@ericwang401

Copy link
Copy Markdown
Collaborator

A PR whose contributors have all signed still got the full CLA comment, signing statement included, so it read as a request to sign (e.g. #171).

  • Passing, no prior comment: posts nothing. The green CLA status is the signal.
  • Passing, prior comment from a failing run: edits it to a short "all signed" note so it stops asking for signatures.
  • Failing: unchanged.

Checked by extracting the script and running node --check. Not exercised live: pull_request_target runs from main, so it takes effect on the first PR after merge.

A pull request whose contributors have all signed got the full bot comment,
signing statement and all, which read as a request to sign. A passing run
now posts nothing; the green CLA status says enough. A comment left by an
earlier failing run is still updated to a short passed note, so it stops
asking for signatures that have since landed.
@github-actions

Copy link
Copy Markdown

Contributor License Agreement

All identified human contributors have signed the current CLA.

Current CLA: https://github.com/ConvoyPanel/panel/blob/47c3288b4e4bf24666b78e76986970baa7b398db/CONTRIBUTOR_LICENSE_AGREEMENT

To sign, comment exactly:

I am at least 18 years old, I have read the Performave Individual Contributor License Agreement, and I electronically sign and agree to it.

This signature is only for individual contributors who are at least 18 years old. If your contribution is owned by a company, employer, client, school, or other legal entity, don't sign here. It needs separate written permission or a separate contributor agreement instead.

Missing signatures:

  • None

Commits without a linked GitHub author, requiring maintainer review:

  • None

If you already signed, comment recheck cla.

@ericwang401
ericwang401 merged commit 290d394 into main Oct 16, 2026
3 checks passed
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