fix: only run CLA check on pull requests
Metadata
Current evaluation
Merged a configuration fix restricting the CLA checker to pull requests only. This eliminates unnecessary failures on main branch pushes, aligns with standard practices, and was approved with a single-line change.
Suggested action: —
No scores available.
Issue body
It doesn't make sense to run the CLA checker once we're already on `main` and it fails anyway.
This returns us back to the [standard method](https://github.com/canonical/has-signed-canonical-cla) which is to just run on PRs.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a configuration fix restricting the CLA checker to pull requests only. This eliminates unnecessary failures on main branch pushes, aligns with standard practices, and was approved with a single-line change. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix restricting the CLA check to pull requests only, preventing unnecessary failures on main branch pushes and restoring the standard signing workflow. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1482 ci: skip the CLA check for pull requests from the bot | snapcraft | merged | Merged a two-line CI configuration update to skip the CLA check for bot-generated pull requests. Approved by a maintainer and integrated into the codebase. | |
| #1115 ci:update the CLA checker workflow | craft-parts | merged | Updating the CLA checker workflow, the pull request was approved by two reviewers, passed all CI checks, and successfully merged. The change modified a single file with two additions and two deletions. | |
| #1503 ci: speedup the CLA check | snapcraft | merged | Merged after approval. The change optimizes the CLA check by verifying emails once and skipping commits already in master. A minor CI failure caused by an outdated git version in the docker image was acknowledged and resolved prior to merge. | |
| #489 ci: add CLA check (CRAFT-305) | charmcraft | merged | Merged. Added a Contributor License Agreement check to the CI workflow. Approved by two reviewers with no unresolved comments. The change introduces nine lines to automate CLA verification. | |
| #64 ci: drop redundant cla check | starflow | merged | Merged removal of a redundant local CLA check, now handled at the organization level. Approved by two reviewers with all CI checks passing. Deleted 7 lines from the CI workflow file. | |
| #625 ci: fix cla failure | fetch-service | merged | Merged after passing all CI checks, including the cla-check, and receiving reviewer approval. The change resolved a continuous integration failure related to contributor license agreement verification. | |
| #112 fix: only run conventional commits check on PRs | starflow | merged | Merged a fix that restricts the conventional commits check to pull requests only. Approved by two reviewers, passed all CI checks, and applied a single-line change to one file. | |
| #3233 travis: set cla check as final stage | snapcraft | merged | Merged a Travis CI configuration update that moves the CLA check to the final stage. This ensures test results display regardless of CLA status, enabling ongoing development while contributors resolve compliance. Approved and passed all checks. |