← 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 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.

Suggested action:

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
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
#1679 ci: remove cla-check from merge groups charmcraft merged Merged a CI change removing the cla-check from merge groups. One line was deleted from a single file to streamline the workflow. No reviews or CI checks were recorded prior to merge.
84%
#1682 ci: no-op step for cla-check if in merge group charmcraft merged Merged a CI configuration update that adds a no-op step to skip the CLA check for merge group workflows. Approved by two reviewers and integrated with a four-line change to a single file.
82%
#128 ci: remove superseded CLA check imagecraft merged Merged removal of the cla-check.yaml CI workflow, superseded by the policy.yaml workflow from starbase. Approved by two reviewers, the change deletes one file and removes nine lines of configuration.
81%
#699 ci: remove direct cla-check workflow craft-application merged Merged removal of the direct cla-check workflow, as its functionality is now covered by the starflow policy workflow. Approved by two reviewers and passing CI checks, the change removes 9 lines from a single file.
81%
#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.
79%
#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.
78%
#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.
78%
#1680 ci: remove cla-check from merge groups charmcraft merged Merged a maintainer change that removed the cla-check from merge groups, updating one file with two lines.
77%
#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.
76%
#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.
74%