← Back to issue list

ci: add CLA check (CRAFT-305)

View original Github issue

Metadata

Project
charmcraft
Number
#489
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #489 added a CI CLA check for CRAFT-305 in commit 6ed6bec1; merge commit: 81a87192.

Resolution: closed resolved

Reason: The CLA check was merged after approval by two reviewers, resolving CRAFT-305 via PR #489 and merge commit 81a87192.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #489 added a CI CLA check for CRAFT-305 in commit 6ed6bec1; merge commit: 81a87192.
stealth/space-bunny-alpha — closed resolved Merged the CRAFT-305 CI change, adding a 9-line CLA check after approval by two reviewers.
stealth/space-bunny-alpha — closed resolved CLA check added to CI in approved, merged PR CRAFT-305 (+9 lines); no unresolved review comments.
qwen/qwen3.6-35b-a3b — — 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.
qwen3.6-35b-a3b-mtp-q6 — — Merged by maintainer sergiusens to add a CLA check to the CI pipeline, resolving CRAFT-305. Integrated without comments or changes.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#625 ci: fix cla failure fetch-service merged Merged PR #625 via commit 5c88af3 to fix CLA-check CI failures; cla-check and all other reported CI checks passed.
77%
#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.
74%
#1503 ci: speedup the CLA check snapcraft merged Merged in snapcraft#1503 (commit 22ef942d6), speeding CLA validation by checking each email once and skipping commits already in master.
74%
#64 ci: drop redundant cla check starflow merged Merged PR #64, removing seven redundant CLA-check lines because Canonical now provides the CLA workflow organization-wide; approvals and all CI checks passed.
73%
#1106 tests: check the CLA on all the commits snapcraft merged Merged PR #1106 updated tests to check the CLA across every commit in a branch; merged as commit 56e934447.
73%
#1680 ci: remove cla-check from merge groups charmcraft merged Merged PR #1680 as commit e2ad204f, updating CI to exclude cla-check from merge-group runs.
72%
#34 fix: only run CLA check on pull requests starflow merged Merged PR #34 in commit ce7ea52, restricting the CLA check to pull requests so it no longer runs and fails on main.
71%
#352 ci: add conditionals for docs-only checks imagecraft merged Merged CRAFT-5161 CI changes in #352 (commit 0104193), adding conditionals to skip required checks and unnecessary operations for docs-only changes.
70%