← Back to issue list

ci: drop redundant cla check

View original Github issue

Metadata

Project
starflow
Number
#64
Type
pull request
State
merged
Author
mr-cal
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #64, removing seven redundant CLA-check lines because Canonical now provides the CLA workflow organization-wide; approvals and all CI checks passed.

Resolution: closed resolved

Reason: PR #64 merged after approval by two reviewers. It removed the duplicate repository-level CLA job in favor of Canonical's organization-wide workflow, and all reported CI checks passed.

Evaluated by: mr-cal-bot

No scores available.

Issue body

The [cla check](https://github.com/canonical/cla-workflow-hosting/blob/93f36d8a49231d6132c203336148ce91335ef7b0/.github/workflows/cla.yml) is now an org-wide workflow, so we don't need to run it twice.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #64, removing seven redundant CLA-check lines because Canonical now provides the CLA workflow organization-wide; approvals and all CI checks passed.
stealth/space-bunny-alpha — closed resolved Merged a CI cleanup removing a duplicate CLA check now covered by Canonical's org-wide CLA workflow; all CI checks passed.
qwen/qwen3.6-35b-a3b — — 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.
qwen3.6-35b-a3b-mtp-q6 — — Merged to remove a redundant CLA check from the CI workflow. The check is now managed organization-wide, eliminating duplicate execution.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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.
79%
#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.
78%
#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.
78%
#128 ci: remove superseded CLA check imagecraft merged Merged PR #128 via commit ff5aba5, removing the obsolete CLA workflow after the policy workflow superseded it.
77%
#778 ci: update cla check craft-providers merged Merged #778 removed the duplicate repository-level CLA workflow, unblocking #777. Upstream policy-checker improvements continue in canonical/starflow#64.
77%
#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.
74%
#699 ci: remove direct cla-check workflow craft-application merged Merged PR #699 removed the redundant direct CLA-check workflow after Starflow’s policy workflow began covering the check; merged as commit 551a49cc.
73%
#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.
73%
#1679 ci: remove cla-check from merge groups charmcraft merged Merged PR #1679 removed cla-check from Charmcraft merge groups with a one-line GitHub Actions configuration change.
72%
#1482 ci: skip the CLA check for pull requests from the bot snapcraft merged Merged maintainer PR #1482 (commit 305b46d34) — a 2-line .travis.yml change skipping the CLA check on pull requests opened by the bot; approved by @sergiusens.
71%