← Back to issue list

ci: remove filter from lint workflow

View original Github issue

Metadata

Project
starflow
Number
#141
Type
pull request
State
merged
Author
medubelko
Labels
—
Created
Updated
Closed

Current evaluation

Merged following two approvals and successful CI checks. Removes the filter from the lint workflow, delegating it to the calling workflow which defaults to affected source files. Resolves part of CRAFT-5161.

Suggested action: —

No scores available.

Issue body

The calling workflow will provide the filter. Otherwise, default to source files being affected. Part of a fix for CRAFT-5161.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged following two approvals and successful CI checks. Removes the filter from the lint workflow, delegating it to the calling workflow which defaults to affected source files. Resolves part of CRAFT-5161.
qwen3.6-35b-a3b-mtp-q6 — — Merged changes remove the lint workflow filter, delegating it to the calling workflow which defaults to affected source files. Addresses CRAFT-5161.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2701 ci: filter in qa workflow charmcraft merged Merged. Adds a filter to the QA workflow and passes it to the lint workflow. Approved by two reviewers. Tests are intentionally skipped to be verified before merge.
75%
#2740 ci: Fix CI filter job and linting issues charmcraft merged Merged changes resolving CI filter job git errors and linting failures. Added a missing checkout step to policy.yaml and removed an unused ty: ignore comment. Approved by two reviewers with all CI checks passing.
73%
#136 ci: filter for .readthedocs.yaml starflow merged Merged after approval and passing CI. Filters CI to trigger only docs lint for .readthedocs.yaml changes. Conflicts resolved by rebasing onto origin/main and updating the lint workflow.
73%
#4018 Remove non-tox CI tests and linting snapcraft merged Merged to remove non-tox CI tests and linting. The change deleted 90 lines in one file, was approved by a reviewer, passed CI checks, and resulted in zero coverage change per Codecov.
70%