ci: speedup the CLA check
Metadata
Current evaluation
Merged in snapcraft#1503 (commit 22ef942d6), speeding CLA validation by checking each email once and skipping commits already in master.
Resolution: closed resolved
Reason: The approved PR was merged as commit 22ef942d6 after implementing the CLA-check optimizations.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Verify an email only once and don't verify commits that are already in master
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged in snapcraft#1503 (commit 22ef942d6), speeding CLA validation by checking each email once and skipping commits already in master. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in snapcraft#1503, speeding up CLA checks by verifying each email once and skipping commits already in master. | |
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to speed up the CLA check by verifying emails once and skipping commits already in master. A CI failure due to an outdated git version in the docker image was resolved, allowing successful integration. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1657571 CLA check only checks latest commit | snapcraft (launchpad) | closed | CLA validation was expanded to check every pull-request commit, not only HEAD; merged in canonical/snapcraft#1106 as 56e934447. | |
| #489 ci: add CLA check (CRAFT-305) | charmcraft | merged | Merged PR #489 added a CI CLA check for CRAFT-305 in commit 6ed6bec1; merge commit: 81a87192. | |
| #3368 ci: use github actions for cla check | snapcraft | merged | Merged in snapcraft#3368 via ead57405c, replacing the previous CLA CI check with a GitHub Actions workflow. | |
| #1649985 We are missing an automatic CLA check for our pull requests | snapcraft (launchpad) | closed | Automatic pull-request CLA verification was added by snapcore/snapcraft#973, merged as commit 9d262f761; Travis CI checked Canonical emails or Launchpad CLA team membership. | |
| #2415 tools: pull in imporved cla_check.py from snapd | snapcraft | merged | Merged in snapcraft#2415 as e32c871cc, replacing tools/cla_check.py with snapd's faster, clearer implementation. | |
| #1115 ci:update the CLA checker workflow | craft-parts | merged | Merged #1115 updated the CLA checker workflow in commit 8d0c1253, with two approvals and all CI checks passing. |